Move some comments

Author: brandtbucher

Python/ceval.c

@@ -884,11 +884,11 @@ _Py_CheckRecursiveCall(PyThreadState *tstate, const char *where)
 // PEP 634: Structural Pattern Matching
 
 
+// Return a tuple of values corresponding to keys, with error checks for
+// duplicate/missing keys.
 static PyObject*
 match_keys(PyThreadState *tstate, PyObject *map, PyObject *keys)
 {
-    // Return a tuple of values corresponding to keys, with error checks for
-    // duplicate/missing keys.
     assert(PyTuple_CheckExact(keys));
     Py_ssize_t nkeys = PyTuple_GET_SIZE(keys);
     if (!nkeys) {
@@ -961,13 +961,13 @@ match_keys(PyThreadState *tstate, PyObject *map, PyObject *keys)
     return NULL;
 }
 
+// Extract a named attribute from the subject, with additional bookkeeping to
+// raise TypeErrors for repeated lookups. On failure, return NULL (with no
+// error set). Use _PyErr_Occurred(tstate) to disambiguate.
 static PyObject*
 match_class_attr(PyThreadState *tstate, PyObject *subject, PyObject *type,
                  PyObject *name, PyObject *seen)
 {
-    // Extract a named attribute from the subject, with additional bookkeeping
-    // to raise TypeErrors for repeated lookups. On failure, return NULL (with
-    // no error set). Use _PyErr_Occurred(tstate) to disambiguate.
     assert(PyUnicode_CheckExact(name));
     assert(PySet_CheckExact(seen));
     if (PySet_Contains(seen, name) || PySet_Add(seen, name)) {
@@ -986,12 +986,12 @@ match_class_attr(PyThreadState *tstate, PyObject *subject, PyObject *type,
     return attr;
 }
 
+// On success (match), return a tuple of extracted attributes. On failure (no
+// match), return NULL. Use _PyErr_Occurred(tstate) to disambiguate.
 static PyObject*
 match_class(PyThreadState *tstate, PyObject *subject, PyObject *type,
             Py_ssize_t nargs, PyObject *kwargs)
 {
-    // On success (match), return a tuple of extracted attributes. On failure
-    // (no match), return NULL. Use _PyErr_Occurred(tstate) to disambiguate.
     if (!PyType_Check(type)) {
         const char *e = "called match pattern must be a type";
         _PyErr_Format(tstate, PyExc_TypeError, e);

Python/compile.c

@@ -5568,14 +5568,13 @@ pattern_helper_sequence_unpack(struct compiler *c, asdl_expr_seq *values,
     return 0;
 }
 
-
+// Like pattern_helper_sequence_unpack, but uses BINARY_SUBSCR instead of
+// UNPACK_SEQUENCE / UNPACK_EX. This is more efficient for patterns with a
+// starred wildcard like [first, *_] / [first, *_, last] / [*_, last] / etc.
 static int
 pattern_helper_sequence_subscr(struct compiler *c, asdl_expr_seq *values,
                                Py_ssize_t star, pattern_context *pc)
 {
-    // Like pattern_helper_sequence_unpack, but uses BINARY_SUBSCR instead of
-    // UNPACK_SEQUENCE / UNPACK_EX. This is more efficient for patterns with a
-    // starred wildcard like [first, *_] / [first, *_, last] / [*_, last] / etc.
     basicblock *end, *fail_pop_1;
     RETURN_IF_FALSE(end = compiler_new_block(c));
     RETURN_IF_FALSE(fail_pop_1 = compiler_new_block(c));
@@ -5617,10 +5616,10 @@ pattern_helper_sequence_subscr(struct compiler *c, asdl_expr_seq *values,
 }
 
 
+// Like compiler_pattern, but turn off checks for irrefutability.
 static int
 compiler_pattern_subpattern(struct compiler *c, expr_ty p, pattern_context *pc)
 {
-    // Like compiler_pattern, but turn off checks for irrefutability.
     int allow_irrefutable = pc->allow_irrefutable;
     pc->allow_irrefutable = 1;
     RETURN_IF_FALSE(compiler_pattern(c, p, pc));