Mercurial > cpython-lru_cache-weakref
annotate Objects/codeobject.c @ 44239:369a4e06fd30
Added another test case for kwonly methods
| author | Christian Heimes <christian@cheimes.de> |
|---|---|
| date | Sat, 08 Dec 2007 22:32:47 +0000 |
| parents | 0d462d789b18 |
| children | b70485011214 |
| rev | line source |
|---|---|
| 35742 | 1 #include "Python.h" |
| 2 #include "code.h" | |
| 3 #include "structmember.h" | |
| 4 | |
| 5 #define NAME_CHARS \ | |
| 6 "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ_abcdefghijklmnopqrstuvwxyz" | |
| 7 | |
| 8 /* all_name_chars(s): true iff all chars in s are valid NAME_CHARS */ | |
| 9 | |
| 10 static int | |
|
43777
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
11 all_name_chars(Py_UNICODE *s) |
| 35742 | 12 { |
| 13 static char ok_name_char[256]; | |
| 14 static unsigned char *name_chars = (unsigned char *)NAME_CHARS; | |
| 15 | |
| 16 if (ok_name_char[*name_chars] == 0) { | |
| 17 unsigned char *p; | |
| 18 for (p = name_chars; *p; p++) | |
| 19 ok_name_char[*p] = 1; | |
| 20 } | |
| 21 while (*s) { | |
|
43777
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
22 if (*s >= 128) |
|
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
23 return 0; |
| 35742 | 24 if (ok_name_char[*s++] == 0) |
| 25 return 0; | |
| 26 } | |
| 27 return 1; | |
| 28 } | |
| 29 | |
| 30 static void | |
| 31 intern_strings(PyObject *tuple) | |
| 32 { | |
|
36501
7fdc639bc5b4
Merge ssize_t branch.
Martin v. Löwis <martin@v.loewis.de>
parents:
35742
diff
changeset
|
33 Py_ssize_t i; |
| 35742 | 34 |
| 35 for (i = PyTuple_GET_SIZE(tuple); --i >= 0; ) { | |
| 36 PyObject *v = PyTuple_GET_ITEM(tuple, i); | |
|
42013
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
37 if (v == NULL || !PyUnicode_CheckExact(v)) { |
| 35742 | 38 Py_FatalError("non-string found in code slot"); |
| 39 } | |
|
42013
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
40 PyUnicode_InternInPlace(&PyTuple_GET_ITEM(tuple, i)); |
| 35742 | 41 } |
| 42 } | |
| 43 | |
| 44 | |
| 45 PyCodeObject * | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
46 PyCode_New(int argcount, int kwonlyargcount, |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
47 int nlocals, int stacksize, int flags, |
| 35742 | 48 PyObject *code, PyObject *consts, PyObject *names, |
| 49 PyObject *varnames, PyObject *freevars, PyObject *cellvars, | |
| 50 PyObject *filename, PyObject *name, int firstlineno, | |
| 51 PyObject *lnotab) | |
| 52 { | |
| 53 PyCodeObject *co; | |
|
36501
7fdc639bc5b4
Merge ssize_t branch.
Martin v. Löwis <martin@v.loewis.de>
parents:
35742
diff
changeset
|
54 Py_ssize_t i; |
|
43569
19319e70fc3e
Patch #1272, by Christian Heimes and Alexandre Vassalotti.
Guido van Rossum <guido@python.org>
parents:
42802
diff
changeset
|
55 |
| 35742 | 56 /* Check argument types */ |
| 57 if (argcount < 0 || nlocals < 0 || | |
| 58 code == NULL || | |
| 59 consts == NULL || !PyTuple_Check(consts) || | |
| 60 names == NULL || !PyTuple_Check(names) || | |
| 61 varnames == NULL || !PyTuple_Check(varnames) || | |
| 62 freevars == NULL || !PyTuple_Check(freevars) || | |
| 63 cellvars == NULL || !PyTuple_Check(cellvars) || | |
|
43569
19319e70fc3e
Patch #1272, by Christian Heimes and Alexandre Vassalotti.
Guido van Rossum <guido@python.org>
parents:
42802
diff
changeset
|
64 name == NULL || !PyUnicode_Check(name) || |
|
19319e70fc3e
Patch #1272, by Christian Heimes and Alexandre Vassalotti.
Guido van Rossum <guido@python.org>
parents:
42802
diff
changeset
|
65 filename == NULL || !PyUnicode_Check(filename) || |
| 35742 | 66 lnotab == NULL || !PyString_Check(lnotab) || |
| 67 !PyObject_CheckReadBuffer(code)) { | |
| 68 PyErr_BadInternalCall(); | |
| 69 return NULL; | |
| 70 } | |
| 71 intern_strings(names); | |
| 72 intern_strings(varnames); | |
| 73 intern_strings(freevars); | |
| 74 intern_strings(cellvars); | |
| 75 /* Intern selected string constants */ | |
| 76 for (i = PyTuple_Size(consts); --i >= 0; ) { | |
| 77 PyObject *v = PyTuple_GetItem(consts, i); | |
|
43777
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
78 if (!PyUnicode_Check(v)) |
| 35742 | 79 continue; |
|
43777
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
80 if (!all_name_chars(PyUnicode_AS_UNICODE(v))) |
| 35742 | 81 continue; |
|
43777
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
82 PyUnicode_InternInPlace(&PyTuple_GET_ITEM(consts, i)); |
| 35742 | 83 } |
| 84 co = PyObject_NEW(PyCodeObject, &PyCode_Type); | |
| 85 if (co != NULL) { | |
| 86 co->co_argcount = argcount; | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
87 co->co_kwonlyargcount = kwonlyargcount; |
| 35742 | 88 co->co_nlocals = nlocals; |
| 89 co->co_stacksize = stacksize; | |
| 90 co->co_flags = flags; | |
| 91 Py_INCREF(code); | |
| 92 co->co_code = code; | |
| 93 Py_INCREF(consts); | |
| 94 co->co_consts = consts; | |
| 95 Py_INCREF(names); | |
| 96 co->co_names = names; | |
| 97 Py_INCREF(varnames); | |
| 98 co->co_varnames = varnames; | |
| 99 Py_INCREF(freevars); | |
| 100 co->co_freevars = freevars; | |
| 101 Py_INCREF(cellvars); | |
| 102 co->co_cellvars = cellvars; | |
| 103 Py_INCREF(filename); | |
| 104 co->co_filename = filename; | |
| 105 Py_INCREF(name); | |
| 106 co->co_name = name; | |
| 107 co->co_firstlineno = firstlineno; | |
| 108 Py_INCREF(lnotab); | |
| 109 co->co_lnotab = lnotab; | |
|
38453
29ae5e0fc348
Much-needed merge (using svnmerge.py this time) of trunk changes into p3yk.
Thomas Wouters <thomas@python.org>
parents:
37879
diff
changeset
|
110 co->co_zombieframe = NULL; |
| 35742 | 111 } |
| 112 return co; | |
| 113 } | |
| 114 | |
| 115 | |
| 116 #define OFF(x) offsetof(PyCodeObject, x) | |
| 117 | |
| 118 static PyMemberDef code_memberlist[] = { | |
| 119 {"co_argcount", T_INT, OFF(co_argcount), READONLY}, | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
120 {"co_kwonlyargcount", T_INT, OFF(co_kwonlyargcount), READONLY}, |
| 35742 | 121 {"co_nlocals", T_INT, OFF(co_nlocals), READONLY}, |
| 122 {"co_stacksize",T_INT, OFF(co_stacksize), READONLY}, | |
| 123 {"co_flags", T_INT, OFF(co_flags), READONLY}, | |
| 124 {"co_code", T_OBJECT, OFF(co_code), READONLY}, | |
| 125 {"co_consts", T_OBJECT, OFF(co_consts), READONLY}, | |
| 126 {"co_names", T_OBJECT, OFF(co_names), READONLY}, | |
| 127 {"co_varnames", T_OBJECT, OFF(co_varnames), READONLY}, | |
| 128 {"co_freevars", T_OBJECT, OFF(co_freevars), READONLY}, | |
| 129 {"co_cellvars", T_OBJECT, OFF(co_cellvars), READONLY}, | |
| 130 {"co_filename", T_OBJECT, OFF(co_filename), READONLY}, | |
| 131 {"co_name", T_OBJECT, OFF(co_name), READONLY}, | |
| 132 {"co_firstlineno", T_INT, OFF(co_firstlineno), READONLY}, | |
| 133 {"co_lnotab", T_OBJECT, OFF(co_lnotab), READONLY}, | |
| 134 {NULL} /* Sentinel */ | |
| 135 }; | |
| 136 | |
| 137 /* Helper for code_new: return a shallow copy of a tuple that is | |
| 138 guaranteed to contain exact strings, by converting string subclasses | |
| 139 to exact strings and complaining if a non-string is found. */ | |
| 140 static PyObject* | |
| 141 validate_and_copy_tuple(PyObject *tup) | |
| 142 { | |
| 143 PyObject *newtuple; | |
| 144 PyObject *item; | |
|
36501
7fdc639bc5b4
Merge ssize_t branch.
Martin v. Löwis <martin@v.loewis.de>
parents:
35742
diff
changeset
|
145 Py_ssize_t i, len; |
| 35742 | 146 |
| 147 len = PyTuple_GET_SIZE(tup); | |
| 148 newtuple = PyTuple_New(len); | |
| 149 if (newtuple == NULL) | |
| 150 return NULL; | |
| 151 | |
| 152 for (i = 0; i < len; i++) { | |
| 153 item = PyTuple_GET_ITEM(tup, i); | |
|
42013
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
154 if (PyUnicode_CheckExact(item)) { |
| 35742 | 155 Py_INCREF(item); |
| 156 } | |
|
42013
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
157 else if (!PyUnicode_Check(item)) { |
| 35742 | 158 PyErr_Format( |
| 159 PyExc_TypeError, | |
| 160 "name tuples must contain only " | |
| 161 "strings, not '%.500s'", | |
| 162 item->ob_type->tp_name); | |
| 163 Py_DECREF(newtuple); | |
| 164 return NULL; | |
| 165 } | |
| 166 else { | |
|
42013
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
167 item = PyUnicode_FromUnicode( |
|
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
168 PyUnicode_AS_UNICODE(item), |
|
cd125fe83051
Make identifiers str (not str8) objects throughout.
Martin v. Löwis <martin@v.loewis.de>
parents:
41962
diff
changeset
|
169 PyUnicode_GET_SIZE(item)); |
| 35742 | 170 if (item == NULL) { |
| 171 Py_DECREF(newtuple); | |
| 172 return NULL; | |
| 173 } | |
| 174 } | |
| 175 PyTuple_SET_ITEM(newtuple, i, item); | |
| 176 } | |
| 177 | |
| 178 return newtuple; | |
| 179 } | |
| 180 | |
| 181 PyDoc_STRVAR(code_doc, | |
| 182 "code(argcount, nlocals, stacksize, flags, codestring, constants, names,\n\ | |
| 183 varnames, filename, name, firstlineno, lnotab[, freevars[, cellvars]])\n\ | |
| 184 \n\ | |
| 185 Create a code object. Not for the faint of heart."); | |
| 186 | |
| 187 static PyObject * | |
| 188 code_new(PyTypeObject *type, PyObject *args, PyObject *kw) | |
| 189 { | |
| 190 int argcount; | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
191 int kwonlyargcount; |
| 35742 | 192 int nlocals; |
| 193 int stacksize; | |
| 194 int flags; | |
| 195 PyObject *co = NULL; | |
| 196 PyObject *code; | |
| 197 PyObject *consts; | |
| 198 PyObject *names, *ournames = NULL; | |
| 199 PyObject *varnames, *ourvarnames = NULL; | |
| 200 PyObject *freevars = NULL, *ourfreevars = NULL; | |
| 201 PyObject *cellvars = NULL, *ourcellvars = NULL; | |
| 202 PyObject *filename; | |
| 203 PyObject *name; | |
| 204 int firstlineno; | |
| 205 PyObject *lnotab; | |
| 206 | |
|
43777
0d462d789b18
Merging the py3k-pep3137 branch back into the py3k branch.
Guido van Rossum <guido@python.org>
parents:
43672
diff
changeset
|
207 if (!PyArg_ParseTuple(args, "iiiiiSO!O!O!UUiS|O!O!:code", |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
208 &argcount, &kwonlyargcount, |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
209 &nlocals, &stacksize, &flags, |
| 35742 | 210 &code, |
| 211 &PyTuple_Type, &consts, | |
| 212 &PyTuple_Type, &names, | |
| 213 &PyTuple_Type, &varnames, | |
| 214 &filename, &name, | |
| 215 &firstlineno, &lnotab, | |
| 216 &PyTuple_Type, &freevars, | |
| 217 &PyTuple_Type, &cellvars)) | |
| 218 return NULL; | |
| 219 | |
| 220 if (argcount < 0) { | |
| 221 PyErr_SetString( | |
| 222 PyExc_ValueError, | |
| 223 "code: argcount must not be negative"); | |
| 224 goto cleanup; | |
| 225 } | |
| 226 | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
227 if (kwonlyargcount < 0) { |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
228 PyErr_SetString( |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
229 PyExc_ValueError, |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
230 "code: kwonlyargcount must not be negative"); |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
231 goto cleanup; |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
232 } |
| 35742 | 233 if (nlocals < 0) { |
| 234 PyErr_SetString( | |
| 235 PyExc_ValueError, | |
| 236 "code: nlocals must not be negative"); | |
| 237 goto cleanup; | |
| 238 } | |
| 239 | |
| 240 ournames = validate_and_copy_tuple(names); | |
| 241 if (ournames == NULL) | |
| 242 goto cleanup; | |
| 243 ourvarnames = validate_and_copy_tuple(varnames); | |
| 244 if (ourvarnames == NULL) | |
| 245 goto cleanup; | |
| 246 if (freevars) | |
| 247 ourfreevars = validate_and_copy_tuple(freevars); | |
| 248 else | |
| 249 ourfreevars = PyTuple_New(0); | |
| 250 if (ourfreevars == NULL) | |
| 251 goto cleanup; | |
| 252 if (cellvars) | |
| 253 ourcellvars = validate_and_copy_tuple(cellvars); | |
| 254 else | |
| 255 ourcellvars = PyTuple_New(0); | |
| 256 if (ourcellvars == NULL) | |
| 257 goto cleanup; | |
| 258 | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
259 co = (PyObject *)PyCode_New(argcount, kwonlyargcount, |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
260 nlocals, stacksize, flags, |
| 35742 | 261 code, consts, ournames, ourvarnames, |
| 262 ourfreevars, ourcellvars, filename, | |
| 263 name, firstlineno, lnotab); | |
| 264 cleanup: | |
| 265 Py_XDECREF(ournames); | |
| 266 Py_XDECREF(ourvarnames); | |
| 267 Py_XDECREF(ourfreevars); | |
| 268 Py_XDECREF(ourcellvars); | |
| 269 return co; | |
| 270 } | |
| 271 | |
| 272 static void | |
| 273 code_dealloc(PyCodeObject *co) | |
| 274 { | |
| 275 Py_XDECREF(co->co_code); | |
| 276 Py_XDECREF(co->co_consts); | |
| 277 Py_XDECREF(co->co_names); | |
| 278 Py_XDECREF(co->co_varnames); | |
| 279 Py_XDECREF(co->co_freevars); | |
| 280 Py_XDECREF(co->co_cellvars); | |
| 281 Py_XDECREF(co->co_filename); | |
| 282 Py_XDECREF(co->co_name); | |
| 283 Py_XDECREF(co->co_lnotab); | |
|
38453
29ae5e0fc348
Much-needed merge (using svnmerge.py this time) of trunk changes into p3yk.
Thomas Wouters <thomas@python.org>
parents:
37879
diff
changeset
|
284 if (co->co_zombieframe != NULL) |
|
29ae5e0fc348
Much-needed merge (using svnmerge.py this time) of trunk changes into p3yk.
Thomas Wouters <thomas@python.org>
parents:
37879
diff
changeset
|
285 PyObject_GC_Del(co->co_zombieframe); |
| 35742 | 286 PyObject_DEL(co); |
| 287 } | |
| 288 | |
| 289 static PyObject * | |
| 290 code_repr(PyCodeObject *co) | |
| 291 { | |
| 292 int lineno = -1; | |
| 293 char *filename = "???"; | |
| 294 | |
| 295 if (co->co_firstlineno != 0) | |
| 296 lineno = co->co_firstlineno; | |
|
43569
19319e70fc3e
Patch #1272, by Christian Heimes and Alexandre Vassalotti.
Guido van Rossum <guido@python.org>
parents:
42802
diff
changeset
|
297 if (co->co_filename && PyUnicode_Check(co->co_filename)) |
|
19319e70fc3e
Patch #1272, by Christian Heimes and Alexandre Vassalotti.
Guido van Rossum <guido@python.org>
parents:
42802
diff
changeset
|
298 filename = PyUnicode_AsString(co->co_filename); |
|
41962
11930dcf9d4f
Use PyUnicode_FromFormat() directly.
Walter Dörwald <walter@livinglogic.de>
parents:
41769
diff
changeset
|
299 return PyUnicode_FromFormat( |
|
42802
01ee6311ba46
Ensure that code object names (co_name) are unicode.
Neal Norwitz <nnorwitz@gmail.com>
parents:
42334
diff
changeset
|
300 "<code object %.100U at %p, file \"%.300s\", line %d>", |
|
01ee6311ba46
Ensure that code object names (co_name) are unicode.
Neal Norwitz <nnorwitz@gmail.com>
parents:
42334
diff
changeset
|
301 co->co_name, co, filename, lineno); |
| 35742 | 302 } |
| 303 | |
|
39644
13d5e5f0e993
Restructure comparison dramatically. There is no longer a default
Guido van Rossum <guido@python.org>
parents:
39442
diff
changeset
|
304 static PyObject * |
|
13d5e5f0e993
Restructure comparison dramatically. There is no longer a default
Guido van Rossum <guido@python.org>
parents:
39442
diff
changeset
|
305 code_richcompare(PyObject *self, PyObject *other, int op) |
| 35742 | 306 { |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
307 PyCodeObject *co, *cp; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
308 int eq; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
309 PyObject *res; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
310 |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
311 if ((op != Py_EQ && op != Py_NE) || |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
312 !PyCode_Check(self) || |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
313 !PyCode_Check(other)) { |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
314 Py_INCREF(Py_NotImplemented); |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
315 return Py_NotImplemented; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
316 } |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
317 |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
318 co = (PyCodeObject *)self; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
319 cp = (PyCodeObject *)other; |
|
39644
13d5e5f0e993
Restructure comparison dramatically. There is no longer a default
Guido van Rossum <guido@python.org>
parents:
39442
diff
changeset
|
320 |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
321 eq = PyObject_RichCompareBool(co->co_name, cp->co_name, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
322 if (eq <= 0) goto unequal; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
323 eq = co->co_argcount == cp->co_argcount; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
324 if (!eq) goto unequal; |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
325 eq = co->co_kwonlyargcount == cp->co_kwonlyargcount; |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
326 if (!eq) goto unequal; |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
327 eq = co->co_nlocals == cp->co_nlocals; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
328 if (!eq) goto unequal; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
329 eq = co->co_flags == cp->co_flags; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
330 if (!eq) goto unequal; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
331 eq = co->co_firstlineno == cp->co_firstlineno; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
332 if (!eq) goto unequal; |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
333 eq = PyObject_RichCompareBool(co->co_code, cp->co_code, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
334 if (eq <= 0) goto unequal; |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
335 eq = PyObject_RichCompareBool(co->co_consts, cp->co_consts, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
336 if (eq <= 0) goto unequal; |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
337 eq = PyObject_RichCompareBool(co->co_names, cp->co_names, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
338 if (eq <= 0) goto unequal; |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
339 eq = PyObject_RichCompareBool(co->co_varnames, cp->co_varnames, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
340 if (eq <= 0) goto unequal; |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
341 eq = PyObject_RichCompareBool(co->co_freevars, cp->co_freevars, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
342 if (eq <= 0) goto unequal; |
|
39658
a5048c8d8f33
Fix a bunch of compiler warnings. In at least one case these were serious bugs!
Guido van Rossum <guido@python.org>
parents:
39650
diff
changeset
|
343 eq = PyObject_RichCompareBool(co->co_cellvars, cp->co_cellvars, Py_EQ); |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
344 if (eq <= 0) goto unequal; |
| 35742 | 345 |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
346 if (op == Py_EQ) |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
347 res = Py_True; |
| 35742 | 348 else |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
349 res = Py_False; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
350 goto done; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
351 |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
352 unequal: |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
353 if (eq < 0) |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
354 return NULL; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
355 if (op == Py_NE) |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
356 res = Py_True; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
357 else |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
358 res = Py_False; |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
359 |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
360 done: |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
361 Py_INCREF(res); |
|
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
362 return res; |
| 35742 | 363 } |
| 364 | |
| 365 static long | |
| 366 code_hash(PyCodeObject *co) | |
| 367 { | |
| 368 long h, h0, h1, h2, h3, h4, h5, h6; | |
| 369 h0 = PyObject_Hash(co->co_name); | |
| 370 if (h0 == -1) return -1; | |
| 371 h1 = PyObject_Hash(co->co_code); | |
| 372 if (h1 == -1) return -1; | |
| 373 h2 = PyObject_Hash(co->co_consts); | |
| 374 if (h2 == -1) return -1; | |
| 375 h3 = PyObject_Hash(co->co_names); | |
| 376 if (h3 == -1) return -1; | |
| 377 h4 = PyObject_Hash(co->co_varnames); | |
| 378 if (h4 == -1) return -1; | |
| 379 h5 = PyObject_Hash(co->co_freevars); | |
| 380 if (h5 == -1) return -1; | |
| 381 h6 = PyObject_Hash(co->co_cellvars); | |
| 382 if (h6 == -1) return -1; | |
| 383 h = h0 ^ h1 ^ h2 ^ h3 ^ h4 ^ h5 ^ h6 ^ | |
|
40215
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
384 co->co_argcount ^ co->co_kwonlyargcount ^ |
|
1cce0526d9b0
Jiwon Seo's PEP 3102 implementation.
Guido van Rossum <guido@python.org>
parents:
39658
diff
changeset
|
385 co->co_nlocals ^ co->co_flags; |
| 35742 | 386 if (h == -1) h = -2; |
| 387 return h; | |
| 388 } | |
| 389 | |
| 390 /* XXX code objects need to participate in GC? */ | |
| 391 | |
| 392 PyTypeObject PyCode_Type = { | |
|
42334
e70cdce0e8b2
Merged revisions 56467-56482 via svnmerge from
Martin v. Löwis <martin@v.loewis.de>
parents:
42013
diff
changeset
|
393 PyVarObject_HEAD_INIT(&PyType_Type, 0) |
| 35742 | 394 "code", |
| 395 sizeof(PyCodeObject), | |
| 396 0, | |
| 397 (destructor)code_dealloc, /* tp_dealloc */ | |
| 398 0, /* tp_print */ | |
| 399 0, /* tp_getattr */ | |
| 400 0, /* tp_setattr */ | |
|
39644
13d5e5f0e993
Restructure comparison dramatically. There is no longer a default
Guido van Rossum <guido@python.org>
parents:
39442
diff
changeset
|
401 0, /* tp_compare */ |
| 35742 | 402 (reprfunc)code_repr, /* tp_repr */ |
| 403 0, /* tp_as_number */ | |
| 404 0, /* tp_as_sequence */ | |
| 405 0, /* tp_as_mapping */ | |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
406 (hashfunc)code_hash, /* tp_hash */ |
| 35742 | 407 0, /* tp_call */ |
| 408 0, /* tp_str */ | |
| 409 PyObject_GenericGetAttr, /* tp_getattro */ | |
| 410 0, /* tp_setattro */ | |
| 411 0, /* tp_as_buffer */ | |
| 412 Py_TPFLAGS_DEFAULT, /* tp_flags */ | |
| 413 code_doc, /* tp_doc */ | |
| 414 0, /* tp_traverse */ | |
| 415 0, /* tp_clear */ | |
|
39650
69f61f90c4fc
Implement == and != comparisons for code objects by value.
Guido van Rossum <guido@python.org>
parents:
39644
diff
changeset
|
416 code_richcompare, /* tp_richcompare */ |
| 35742 | 417 0, /* tp_weaklistoffset */ |
| 418 0, /* tp_iter */ | |
| 419 0, /* tp_iternext */ | |
| 420 0, /* tp_methods */ | |
| 421 code_memberlist, /* tp_members */ | |
| 422 0, /* tp_getset */ | |
| 423 0, /* tp_base */ | |
| 424 0, /* tp_dict */ | |
| 425 0, /* tp_descr_get */ | |
| 426 0, /* tp_descr_set */ | |
| 427 0, /* tp_dictoffset */ | |
| 428 0, /* tp_init */ | |
| 429 0, /* tp_alloc */ | |
| 430 code_new, /* tp_new */ | |
| 431 }; | |
| 432 | |
| 433 /* All about c_lnotab. | |
| 434 | |
| 435 c_lnotab is an array of unsigned bytes disguised as a Python string. In -O | |
| 436 mode, SET_LINENO opcodes aren't generated, and bytecode offsets are mapped | |
| 437 to source code line #s (when needed for tracebacks) via c_lnotab instead. | |
| 438 The array is conceptually a list of | |
| 439 (bytecode offset increment, line number increment) | |
| 440 pairs. The details are important and delicate, best illustrated by example: | |
| 441 | |
| 442 byte code offset source code line number | |
| 443 0 1 | |
| 444 6 2 | |
| 445 50 7 | |
| 446 350 307 | |
| 447 361 308 | |
| 448 | |
| 449 The first trick is that these numbers aren't stored, only the increments | |
| 450 from one row to the next (this doesn't really work, but it's a start): | |
| 451 | |
| 452 0, 1, 6, 1, 44, 5, 300, 300, 11, 1 | |
| 453 | |
| 454 The second trick is that an unsigned byte can't hold negative values, or | |
| 455 values larger than 255, so (a) there's a deep assumption that byte code | |
| 456 offsets and their corresponding line #s both increase monotonically, and (b) | |
| 457 if at least one column jumps by more than 255 from one row to the next, more | |
| 458 than one pair is written to the table. In case #b, there's no way to know | |
| 459 from looking at the table later how many were written. That's the delicate | |
| 460 part. A user of c_lnotab desiring to find the source line number | |
| 461 corresponding to a bytecode address A should do something like this | |
| 462 | |
| 463 lineno = addr = 0 | |
| 464 for addr_incr, line_incr in c_lnotab: | |
| 465 addr += addr_incr | |
| 466 if addr > A: | |
| 467 return lineno | |
| 468 lineno += line_incr | |
| 469 | |
| 470 In order for this to work, when the addr field increments by more than 255, | |
| 471 the line # increment in each pair generated must be 0 until the remaining addr | |
| 472 increment is < 256. So, in the example above, com_set_lineno should not (as | |
| 473 was actually done until 2.2) expand 300, 300 to 255, 255, 45, 45, but to | |
| 474 255, 0, 45, 255, 0, 45. | |
| 475 */ | |
| 476 | |
| 477 int | |
| 478 PyCode_Addr2Line(PyCodeObject *co, int addrq) | |
| 479 { | |
| 480 int size = PyString_Size(co->co_lnotab) / 2; | |
| 481 unsigned char *p = (unsigned char*)PyString_AsString(co->co_lnotab); | |
| 482 int line = co->co_firstlineno; | |
| 483 int addr = 0; | |
| 484 while (--size >= 0) { | |
| 485 addr += *p++; | |
| 486 if (addr > addrq) | |
| 487 break; | |
| 488 line += *p++; | |
| 489 } | |
| 490 return line; | |
| 491 } | |
|
37879
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
492 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
493 /* |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
494 Check whether the current instruction is at the start of a line. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
495 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
496 */ |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
497 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
498 /* The theory of SET_LINENO-less tracing. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
499 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
500 In a nutshell, we use the co_lnotab field of the code object |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
501 to tell when execution has moved onto a different line. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
502 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
503 As mentioned above, the basic idea is so set things up so |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
504 that |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
505 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
506 *instr_lb <= frame->f_lasti < *instr_ub |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
507 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
508 is true so long as execution does not change lines. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
509 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
510 This is all fairly simple. Digging the information out of |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
511 co_lnotab takes some work, but is conceptually clear. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
512 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
513 Somewhat harder to explain is why we don't *always* call the |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
514 line trace function when the above test fails. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
515 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
516 Consider this code: |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
517 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
518 1: def f(a): |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
519 2: if a: |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
520 3: print 1 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
521 4: else: |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
522 5: print 2 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
523 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
524 which compiles to this: |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
525 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
526 2 0 LOAD_FAST 0 (a) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
527 3 JUMP_IF_FALSE 9 (to 15) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
528 6 POP_TOP |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
529 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
530 3 7 LOAD_CONST 1 (1) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
531 10 PRINT_ITEM |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
532 11 PRINT_NEWLINE |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
533 12 JUMP_FORWARD 6 (to 21) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
534 >> 15 POP_TOP |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
535 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
536 5 16 LOAD_CONST 2 (2) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
537 19 PRINT_ITEM |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
538 20 PRINT_NEWLINE |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
539 >> 21 LOAD_CONST 0 (None) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
540 24 RETURN_VALUE |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
541 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
542 If 'a' is false, execution will jump to instruction at offset |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
543 15 and the co_lnotab will claim that execution has moved to |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
544 line 3. This is at best misleading. In this case we could |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
545 associate the POP_TOP with line 4, but that doesn't make |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
546 sense in all cases (I think). |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
547 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
548 What we do is only call the line trace function if the co_lnotab |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
549 indicates we have jumped to the *start* of a line, i.e. if the |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
550 current instruction offset matches the offset given for the |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
551 start of a line by the co_lnotab. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
552 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
553 This also takes care of the situation where 'a' is true. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
554 Execution will jump from instruction offset 12 to offset 21. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
555 Then the co_lnotab would imply that execution has moved to line |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
556 5, which is again misleading. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
557 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
558 Why do we set f_lineno when tracing? Well, consider the code |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
559 above when 'a' is true. If stepping through this with 'n' in |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
560 pdb, you would stop at line 1 with a "call" type event, then |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
561 line events on lines 2 and 3, then a "return" type event -- but |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
562 you would be shown line 5 during this event. This is a change |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
563 from the behaviour in 2.2 and before, and I've found it |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
564 confusing in practice. By setting and using f_lineno when |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
565 tracing, one can report a line number different from that |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
566 suggested by f_lasti on this one occasion where it's desirable. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
567 */ |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
568 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
569 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
570 int |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
571 PyCode_CheckLineNumber(PyCodeObject* co, int lasti, PyAddrPair *bounds) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
572 { |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
573 int size, addr, line; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
574 unsigned char* p; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
575 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
576 p = (unsigned char*)PyString_AS_STRING(co->co_lnotab); |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
577 size = PyString_GET_SIZE(co->co_lnotab) / 2; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
578 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
579 addr = 0; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
580 line = co->co_firstlineno; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
581 assert(line > 0); |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
582 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
583 /* possible optimization: if f->f_lasti == instr_ub |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
584 (likely to be a common case) then we already know |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
585 instr_lb -- if we stored the matching value of p |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
586 somwhere we could skip the first while loop. */ |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
587 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
588 /* see comments in compile.c for the description of |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
589 co_lnotab. A point to remember: increments to p |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
590 should come in pairs -- although we don't care about |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
591 the line increments here, treating them as byte |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
592 increments gets confusing, to say the least. */ |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
593 |
|
39442
654c380cf8b9
Merged revisions 46753-51188 via svnmerge from
Thomas Wouters <thomas@python.org>
parents:
38453
diff
changeset
|
594 bounds->ap_lower = 0; |
|
37879
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
595 while (size > 0) { |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
596 if (addr + *p > lasti) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
597 break; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
598 addr += *p++; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
599 if (*p) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
600 bounds->ap_lower = addr; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
601 line += *p++; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
602 --size; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
603 } |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
604 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
605 /* If lasti and addr don't match exactly, we don't want to |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
606 change the lineno slot on the frame or execute a trace |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
607 function. Return -1 instead. |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
608 */ |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
609 if (addr != lasti) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
610 line = -1; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
611 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
612 if (size > 0) { |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
613 while (--size >= 0) { |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
614 addr += *p++; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
615 if (*p++) |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
616 break; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
617 } |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
618 bounds->ap_upper = addr; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
619 } |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
620 else { |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
621 bounds->ap_upper = INT_MAX; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
622 } |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
623 |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
624 return line; |
|
f731859e9b3b
Merge p3yk branch with the trunk up to revision 45595. This breaks a fair
Thomas Wouters <thomas@python.org>
parents:
36501
diff
changeset
|
625 } |
