jslint && cppcheck

jsteemann · jsteemann · commit e495b5fb6cb0 · 2017-03-06T16:13:04.000+01:00
diff --git a/arangod/GeneralServer/VppCommTask.cpp b/arangod/GeneralServer/VppCommTask.cpp
@@ -56,9 +56,8 @@ inline std::size_t validateAndCount(char const* vpStart,
   validationOptions.validateUtf8Strings = true;
   VPackValidator validator(&validationOptions);
 
-  std::size_t numPayloads = 0;
-
   try {
+    std::size_t numPayloads = 0;
     // check for slice start to the end of Chunk
     // isSubPart allows the slice to be shorter than the checked buffer.
     do {
diff --git a/js/server/tests/recovery/collection-duplicate-name.js b/js/server/tests/recovery/collection-duplicate-name.js
@@ -35,16 +35,14 @@ var jsunity = require('jsunity');
 
 
 function runSetup () {
-  require("console").log("Hund");
-
   db._drop('UnitTestsRecovery');
   db._create('UnitTestsRecovery');
 
   try {
     db._create('UnitTestsRecovery');
   } catch (e) {
     // This intentionally should fail!
-    if (internal.errors.ERROR_ARANGO_DUPLICATE_NAME.code == e.errorNum) {
+    if (internal.errors.ERROR_ARANGO_DUPLICATE_NAME.code === e.errorNum) {
       // Only this is a valid return code from the server
       return 0;
     }
@@ -62,9 +60,8 @@ function recoverySuite () {
     tearDown: function () {},
 
     testCollectionDuplicateName: function () {
-      require("console").log("Kartze");
       var c = db._collection('UnitTestsRecovery');
-      assertTrue(c != null && c != undefined);
+      assertTrue(c !== null && c !== undefined);
     }
   };
 };
