Try to fix static code analysis issues.

src/mlpack/tests/cf_test.cpp

@@ -346,7 +346,7 @@ void TrainWithCoordinateList(DecompositionPolicy& decomposition)
 {
   arma::mat randomData(3, 100);
   randomData.row(0) = arma::linspace<arma::rowvec>(0, 99, 100);
-  randomData.row(1) = arma::linspace<arma::rowvec>(0, 99, 100);
+  randomData.row(1) = randomData.row(0);
   randomData.row(2).fill(3);
   CFType<DecompositionPolicy> c(randomData, decomposition, 5, 5, 30);
 

src/mlpack/tests/rnn_reber_test.cpp

@@ -179,7 +179,6 @@ void GenerateNextRecursiveReber(const arma::Mat<char>& transitions,
     else if (c == 'P' && state == 1)
     {
       numPs++;
-      state = 1;
     }
     else if (c == 'T' && state == 1)
     {
@@ -206,7 +205,6 @@ void GenerateNextRecursiveReber(const arma::Mat<char>& transitions,
     else if (c == 'P' && state == 5)
     {
       numPs--;
-      state = 5;
     }
   }
 

src/mlpack/tests/serialization_test.cpp

@@ -1682,7 +1682,7 @@ TEST_CASE("CerealEmptyArrayWrapperTest", "[SerializationTest]")
   jsonT.mem = new int[5];
   jsonT.len = 5;
 
-  SerializeObjectAll(t, xmlT, binaryT, jsonT);
+  SerializeObjectAll(t, xmlT, jsonT, binaryT);
 
   // Ensure that all the results are correct.
   REQUIRE(xmlT.mem == (int*) NULL);