diff options
| author | Larry Hastings <larry@hastings.org> | 2012-05-08 23:52:03 -0700 |
|---|---|---|
| committer | Larry Hastings <larry@hastings.org> | 2012-05-08 23:52:03 -0700 |
| commit | bca3a7bbad8094ef00817dec65f1592701dc6151 (patch) | |
| tree | f8fe1f537a1e6459c6c521dfbde6552d56b59214 /Python/getargs.c | |
| parent | ba5f2f6a326a6e15777b99b81ace22f962ef55dc (diff) | |
| download | cpython-bca3a7bbad8094ef00817dec65f1592701dc6151.tar.gz | |
Issue #14746: Remove redundant paragraphs from skipitem() in Python/getargs.c.
Diffstat (limited to 'Python/getargs.c')
| -rw-r--r-- | Python/getargs.c | 26 |
1 files changed, 8 insertions, 18 deletions
diff --git a/Python/getargs.c b/Python/getargs.c index 5ddf236aa7..17c4ee5955 100644 --- a/Python/getargs.c +++ b/Python/getargs.c @@ -1609,8 +1609,10 @@ skipitem(const char **p_format, va_list *p_va, int flags) switch (c) { - /* simple codes - * The individual types (second arg of va_arg) are irrelevant */ + /* + * codes that take a single data pointer as an argument + * (the type of the pointer is irrelevant) + */ case 'b': /* byte -- very short int */ case 'B': /* byte as bitfield */ @@ -1624,23 +1626,21 @@ skipitem(const char **p_format, va_list *p_va, int flags) case 'L': /* PY_LONG_LONG */ case 'K': /* PY_LONG_LONG sized bitfield */ #endif + case 'n': /* Py_ssize_t */ case 'f': /* float */ case 'd': /* double */ case 'D': /* complex double */ case 'c': /* char */ case 'C': /* unicode char */ case 'p': /* boolean predicate */ + case 'S': /* string object */ + case 'Y': /* string object */ + case 'U': /* unicode string object */ { (void) va_arg(*p_va, void *); break; } - case 'n': /* Py_ssize_t */ - { - (void) va_arg(*p_va, Py_ssize_t *); - break; - } - /* string codes */ case 'e': /* string with encoding */ @@ -1673,16 +1673,6 @@ skipitem(const char **p_format, va_list *p_va, int flags) break; } - /* object codes */ - - case 'S': /* string object */ - case 'Y': /* string object */ - case 'U': /* unicode string object */ - { - (void) va_arg(*p_va, PyObject **); - break; - } - case 'O': /* object */ { if (*format == '!') { |
