From: Andrew Kuchling <amk@amk.ca>
Date: Mon, 14 Apr 2014 18:19:52 +0000 (-0400)
Subject: #11983: update comment to describe which fields are used and why.
X-Git-Tag: v3.5.0a1~1886
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6283691d461ab3d378d56e9a23a1499a8ad2e655;p=python

#11983: update comment to describe which fields are used and why.

Original patch by Caelyn McAulay; modified after discussion w/ her at
the PyCon 2014 sprints.
---

diff --git a/Include/code.h b/Include/code.h
index 7c7e5bf8dc..ff2b97efe9 100644
--- a/Include/code.h
+++ b/Include/code.h
@@ -21,7 +21,12 @@ typedef struct {
     PyObject *co_varnames;	/* tuple of strings (local variable names) */
     PyObject *co_freevars;	/* tuple of strings (free variable names) */
     PyObject *co_cellvars;      /* tuple of strings (cell variable names) */
-    /* The rest doesn't count for hash or comparisons */
+    /* The rest aren't used in either hash or comparisons, except for
+       co_name (used in both) and co_firstlineno (used only in
+       comparisons).  This is done to preserve the name and line number
+       for tracebacks and debuggers; otherwise, constant de-duplication
+       would collapse identical functions/lambdas defined on different lines.
+    */
     unsigned char *co_cell2arg; /* Maps cell vars which are arguments. */
     PyObject *co_filename;	/* unicode (where it was loaded from) */
     PyObject *co_name;		/* unicode (name, for reference) */