Fix lint errors.

tests/cpp/operator/dropout_perf.cc

@@ -42,8 +42,9 @@ TEST(DROPOUT_PERF, ExecuteBidirectional) {
   kwargs_t kwargs = basic_dropout_args;
   kwargs.push_back({"mode", "always"});
   test::op::CoreOperatorRunner<float> runner;
-  runner.RunGenericOperatorForward(false, { shape }, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-          kwargs, "Dropout", "_backward_Dropout"), 1);
+  kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "Dropout",
+                                                           "_backward_Dropout");
+  runner.RunGenericOperatorForward(false, { shape }, kwargs, 1);
 }
 
 /*!
@@ -55,8 +56,9 @@ TEST(DROPOUT_PERF, TimingCPU) {
   kwargs.push_back({"mode", "always"});
   TShape shape({10, 10, 10, 10});
   test::op::CoreOperatorRunner<float> runner;
-  runner.RunGenericOperatorForward(false, { shape }, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-          kwargs, "Dropout", "_backward_Dropout"), 1);
+  kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "Dropout",
+                                                           "_backward_Dropout");
+  runner.RunGenericOperatorForward(false, { shape }, kwargs, 1);
   std::vector <TShape> shapes;
   if (test::performance_run) {
     shapes = {
@@ -73,8 +75,9 @@ TEST(DROPOUT_PERF, TimingCPU) {
     };
   }
   for (const TShape &shape : shapes) {
-    runner.TimingTest("Dropout Operator CPU", false, false, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-                      kwargs, "Dropout", "_backward_Dropout"), 2, 10, { shape }, false);
+    kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "Dropout",
+                                                             "_backward_Dropout");
+    runner.TimingTest("Dropout Operator CPU", false, false, kwargs, 2, 10, { shape }, false);
   }
 }
 
@@ -88,8 +91,9 @@ TEST(DROPOUT_PERF, TimingGPU) {
   kwargs.push_back({"mode", "always"});
   TShape shape({10, 10, 10, 10});
   test::op::CoreOperatorRunner<float> runner;
-  runner.RunGenericOperatorForward(true, { shape }, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-          kwargs, "Dropout", "_backward_Dropout"), 1);
+  kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "Dropout",
+                                                           "_backward_Dropout");
+  runner.RunGenericOperatorForward(true, { shape }, kwargs, 1);
   std::vector <TShape> shapes = {
     {1,  1, 28,  28},
     {1,  3, 28,  28},
@@ -98,8 +102,9 @@ TEST(DROPOUT_PERF, TimingGPU) {
     {20, 3, 128, 128}
   };
   for (const TShape &shape : shapes) {
-    runner.TimingTest("Dropout Operator GPU", true, false, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-                      kwargs, "Dropout", "_backward_Dropout"), 2, 10, { shape }, false);
+    kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "Dropout",
+                                                             "_backward_Dropout");
+    runner.TimingTest("Dropout Operator GPU", true, false, kwargs, 2, 10, { shape }, false);
   }
 }
 #endif  // MXNET_USE_CUDA == 1

tests/cpp/operator/fully_conn_perf.cc

@@ -45,8 +45,9 @@ TEST(FULLY_CONNECTED, ExecuteBidirectionalFullyConnected) {
   kwargs_t kwargs = basic_fullyconn_args;
   test::op::CoreOperatorRunner<float> runner;
   runner.set_verbose(true);
-  runner.RunGenericOperatorForward(false, { shape1, shape2 }, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-          kwargs, "FullyConnected", "_backward_FullyConnected"), 1);
+  kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "FullyConnected",
+                                                           "_backward_FullyConnected");
+  runner.RunGenericOperatorForward(false, { shape1, shape2 }, kwargs, 1);
 }
 
 /*!
@@ -57,8 +58,9 @@ TEST(FULLY_CONNECTED, FullyConnectedTimingCPU) {
   TShape shape1({10, 10, 10, 10});
   TShape shape2({250, 1000});
   test::op::CoreOperatorRunner<float> runner;
-  runner.RunGenericOperatorForward(false, { shape1, shape2 }, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-          kwargs, "FullyConnected", "_backward_FullyConnected"), 1);
+  kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "FullyConnected",
+                                                           "_backward_FullyConnected");
+  runner.RunGenericOperatorForward(false, { shape1, shape2 }, kwargs, 1);
   std::vector <TShape> shapes;
   if (test::performance_run) {
     shapes = {
@@ -76,8 +78,10 @@ TEST(FULLY_CONNECTED, FullyConnectedTimingCPU) {
   }
   for (const TShape& shape : shapes) {
     TShape shape2({250, shape.ProdShape(1, shape.ndim())});
-    runner.TimingTest("Fully connected CPU", false, false, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-                      kwargs, "FullyConnected", "_backward_FullyConnected"), 2, 10, { shape, shape2 }, false);
+    kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "FullyConnected",
+                                                             "_backward_FullyConnected");
+    runner.TimingTest("Fully connected CPU", false, false, kwargs, 2, 10,
+                      { shape, shape2 }, false);
   }
 }
 
@@ -90,8 +94,9 @@ TEST(FULLY_CONNECTED, FullyConnectedTimingGPU) {
   TShape shape1({10, 10, 10, 10});
   TShape shape2({250, 1000});
   test::op::CoreOperatorRunner<float> runner;
-  runner.RunGenericOperatorForward(true, { shape1, shape2 }, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-          kwargs, "FullyConnected", "_backward_FullyConnected"), 1);
+  kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "FullyConnected",
+                                                           "_backward_FullyConnected");
+  runner.RunGenericOperatorForward(true, { shape1, shape2 }, kwargs, 1);
   std::vector <TShape> shapes;
   if (test::performance_run) {
     shapes = {
@@ -109,8 +114,10 @@ TEST(FULLY_CONNECTED, FullyConnectedTimingGPU) {
   }
   for (const TShape& shape : shapes) {
     TShape shape2({250, shape.ProdShape(1, shape.ndim())});
-    runner.TimingTest("Fully connected GPU", true, false, test::op::CoreOpExecutor<float>::ArgsWithOpName(
-                      kwargs, "FullyConnected", "_backward_FullyConnected"), 2, 10, { shape, shape2 }, false);
+    kwargs = test::op::CoreOpExecutor<float>::ArgsWithOpName(kwargs, "FullyConnected",
+                                                             "_backward_FullyConnected");
+    runner.TimingTest("Fully connected GPU", true, false, kwargs, 2, 10,
+                      { shape, shape2 }, false);
   }
 }
 #endif  // MXNET_USE_CUDA == 1