Fix argument comment in cinderx/RuntimeTests/bytecode_test.cpp +3

Summary:
Argument comments ensure that the correct values are assigned to the correct arguments.

This diff is a manual fix (assisted by the script in D78190897) of one or more argument comments in cinderx/RuntimeTests/bytecode_test.cpp. Once existing violations are drawn down we will make mismatches between argument comments and argument names an error.

Reviewed By: dtolnay

Differential Revision: D78191313

fbshipit-source-id: 568626d6abadb1d6577b5f100ab536f60db9c621

Author: r-barnes

RuntimeTests/bytecode_test.cpp

@@ -41,7 +41,7 @@ TEST_F(BytecodeInstructionIteratorTest, ConsumesExtendedArgs) {
   auto empty_bytes = Ref<>::steal(PyBytes_FromString(""));
   auto code = Ref<PyCodeObject>::steal(PyUnstable_Code_New(
       /*argcount=*/0,
-      /*kwargcount=*/0,
+      /*kwonlyargcount=*/0,
       /*nlocals=*/0,
       /*stacksize=*/0,
       /*flags=*/0,
@@ -53,10 +53,10 @@ TEST_F(BytecodeInstructionIteratorTest, ConsumesExtendedArgs) {
       /*cellvars=*/empty_tuple,
       filename,
       funcname,
-      /*qualname=*/funcname,
+      /*_unused_qualname=*/funcname,
       /*firstlineno=*/0,
       /*linetable=*/empty_bytes,
-      /*exceptiontable=*/empty_bytes));
+      /*_unused_exceptiontable=*/empty_bytes));
   ASSERT_NE(code.get(), nullptr);
 
   jit::BytecodeInstructionBlock bc_block{code};

RuntimeTests/gen_asm_test.cpp

@@ -1343,7 +1343,7 @@ TEST_F(ASMGeneratorTest, GetLength) {
   auto empty_string = Ref<>::steal(PyBytes_FromString(""));
   auto code = Ref<PyCodeObject>::steal(PyUnstable_Code_New(
       /*argcount=*/1,
-      /*kwargcount=*/0,
+      /*kwonlyargcount=*/0,
       /*nlocals=*/1,
       /*stacksize=*/0,
       /*flags=*/0,
@@ -1355,10 +1355,10 @@ TEST_F(ASMGeneratorTest, GetLength) {
       /*cellvars=*/empty_tuple,
       filename,
       funcname,
-      /*qualname=*/funcname,
+      /*_unused_qualname=*/funcname,
       /*firstlineno=*/0,
       /*linetable=*/empty_string,
-      /*exceptiontable=*/empty_string));
+      /*_unused_exceptiontable=*/empty_string));
   ASSERT_NE(code.get(), nullptr);
 
   auto func = Ref<PyFunctionObject>::steal(PyFunction_New(code, MakeGlobals()));

RuntimeTests/hir_test.cpp

@@ -554,10 +554,10 @@ class HIRBuildTest : public RuntimeTest {
         /*cellvars=*/empty_tuple,
         filename,
         funcname,
-        /*qualname=*/funcname,
+        /*_unused_qualname=*/funcname,
         /*firstlineno=*/0,
         /*linetable=*/empty_bytes,
-        /*exceptiontable=*/empty_bytes));
+        /*_unused_exceptiontable=*/empty_bytes));
     assert(code != nullptr);
 
     auto func =
@@ -662,10 +662,10 @@ TEST_F(HIRBuildTest, LoadAssertionError) {
       /*cellvars=*/empty_tuple,
       filename,
       funcname,
-      /*qualname=*/funcname,
+      /*_unused_qualname=*/funcname,
       /*firstlineno=*/0,
       /*linetable=*/empty_bytes,
-      /*exceptiontable=*/empty_bytes));
+      /*_unused_exceptiontable=*/empty_bytes));
   ASSERT_NE(code.get(), nullptr);
 
   auto func = Ref<PyFunctionObject>::steal(PyFunction_New(code, MakeGlobals()));
@@ -755,10 +755,10 @@ TEST_F(HIRBuildTest, SetUpdate) {
       /*cellvars=*/empty_tuple,
       filename,
       funcname,
-      /*qualname=*/funcname,
+      /*_unused_qualname=*/funcname,
       /*firstlineno=*/0,
       /*linetable=*/empty_bytes,
-      /*exceptiontable=*/empty_bytes));
+      /*_unused_exceptiontable=*/empty_bytes));
   ASSERT_NE(code.get(), nullptr);
 
   auto func = Ref<PyFunctionObject>::steal(PyFunction_New(code, MakeGlobals()));
@@ -892,10 +892,10 @@ TEST_F(EdgeCaseTest, IgnoreUnreachableLoops) {
       /*cellvars=*/empty_tuple,
       filename,
       funcname,
-      /*qualname=*/funcname,
+      /*_unused_qualname=*/funcname,
       /*firstlineno=*/0,
       /*linetable=*/empty_bytes,
-      /*exceptiontable=*/empty_bytes));
+      /*_unused_exceptiontable=*/empty_bytes));
   ASSERT_NE(code.get(), nullptr);
 
   auto func = Ref<PyFunctionObject>::steal(PyFunction_New(code, MakeGlobals()));
@@ -976,10 +976,10 @@ TEST_F(EdgeCaseTest, JumpBackwardNoInterrupt) {
       /*cellvars=*/empty_tuple,
       filename,
       funcname,
-      /*qualname=*/funcname,
+      /*_unused_qualname=*/funcname,
       /*firstlineno=*/0,
       /*linetable=*/empty_bytes,
-      /*exceptiontable=*/empty_bytes));
+      /*_unused_exceptiontable=*/empty_bytes));
   ASSERT_NE(code.get(), nullptr);
 
   auto func = Ref<PyFunctionObject>::steal(PyFunction_New(code, MakeGlobals()));