diff options
| author | Neal Norwitz <nnorwitz@gmail.com> | 2006-03-20 01:53:23 (GMT) |
|---|---|---|
| committer | Neal Norwitz <nnorwitz@gmail.com> | 2006-03-20 01:53:23 (GMT) |
| commit | 2aa9a5dfdd2966c57036dc836ba8e91ad47ecf14 (patch) | |
| tree | 49688fd8e420b9a1625345de6593e147523b7b66 /Objects/stringobject.c | |
| parent | 70f05c5d7f0259d056dbb19bc5632c8357fd6998 (diff) | |
| download | cpython-2aa9a5dfdd2966c57036dc836ba8e91ad47ecf14.zip cpython-2aa9a5dfdd2966c57036dc836ba8e91ad47ecf14.tar.gz cpython-2aa9a5dfdd2966c57036dc836ba8e91ad47ecf14.tar.bz2 | |
Use macro versions instead of function versions when we already know the type.
This will hopefully get rid of some Coverity warnings, be a hint to
developers, and be marginally faster.
Some asserts were added when the type is currently known, but depends
on values from another function.
Diffstat (limited to 'Objects/stringobject.c')
| -rw-r--r-- | Objects/stringobject.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/Objects/stringobject.c b/Objects/stringobject.c index d23c973..e3a0197 100644 --- a/Objects/stringobject.c +++ b/Objects/stringobject.c @@ -569,8 +569,9 @@ PyObject *PyString_DecodeEscape(const char *s, if (!w) goto failed; /* Append bytes to output buffer. */ - r = PyString_AsString(w); - rn = PyString_Size(w); + assert(PyString_Check(w)); + r = PyString_AS_STRING(w); + rn = PyString_GET_SIZE(w); memcpy(p, r, rn); p += rn; Py_DECREF(w); @@ -2314,12 +2315,12 @@ string_translate(PyStringObject *self, PyObject *args) } table = table1; - inlen = PyString_Size(input_obj); + inlen = PyString_GET_SIZE(input_obj); result = PyString_FromStringAndSize((char *)NULL, inlen); if (result == NULL) return NULL; output_start = output = PyString_AsString(result); - input = PyString_AsString(input_obj); + input = PyString_AS_STRING(input_obj); if (dellen == 0) { /* If no deletions are required, use faster code */ |