traverseda
/
cpython
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Cleanup unicodeobject.c 

* Remove micro-optization:
   (errors == "surrogateescape" || strcmp(errors, "surrogateescape") == 0).
   Only use strcmp()
 * Initialize 'arg' members in unicode_format_arg() to help the compiler to
   diagnose real bugs and also make the code simpler to read
This commit is contained in:
Victor Stinner 2012-12-04 09:30:24 +01:00
parent d45c7f8d74
commit 8dbd421b4d
1 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
Objects/unicodeobject.c
View File

@ -3110,8 +3110,7 @@ locale_error_handler(const char *errors, int *surrogateescape)
*surrogateescape = 0; *surrogateescape = 0;
return 0; return 0;
} }
if (errors == "surrogateescape" if (strcmp(errors, "surrogateescape") == 0) {
|| strcmp(errors, "surrogateescape") == 0) {
*surrogateescape = 1; *surrogateescape = 1;
return 0; return 0;
} }
@ -13438,7 +13437,6 @@ unicode_format_arg_parse(struct unicode_formatter_t *ctx,
PyObject *v; PyObject *v;
arg->ch = FORMAT_READ(ctx);
if (arg->ch == '(') { if (arg->ch == '(') {
/* Get argument value from a dictionary. Example: "%(name)s". */ /* Get argument value from a dictionary. Example: "%(name)s". */
Py_ssize_t keystart; Py_ssize_t keystart;
@ -13487,7 +13485,6 @@ unicode_format_arg_parse(struct unicode_formatter_t *ctx,
} }
/* Parse flags. Example: "%+i" => flags=F_SIGN. */ /* Parse flags. Example: "%+i" => flags=F_SIGN. */
arg->flags = 0;
while (--ctx->fmtcnt >= 0) { while (--ctx->fmtcnt >= 0) {
arg->ch = FORMAT_READ(ctx); arg->ch = FORMAT_READ(ctx);
ctx->fmtpos++; ctx->fmtpos++;
@ -13502,7 +13499,6 @@ unicode_format_arg_parse(struct unicode_formatter_t *ctx,
} }
/* Parse width. Example: "%10s" => width=10 */ /* Parse width. Example: "%10s" => width=10 */
arg->width = -1;
if (arg->ch == '*') { if (arg->ch == '*') {
v = unicode_format_getnextarg(ctx); v = unicode_format_getnextarg(ctx);
if (v == NULL) if (v == NULL)
@ -13544,7 +13540,6 @@ unicode_format_arg_parse(struct unicode_formatter_t *ctx,
} }
/* Parse precision. Example: "%.3f" => prec=3 */ /* Parse precision. Example: "%.3f" => prec=3 */
arg->prec = -1;
if (arg->ch == '.') { if (arg->ch == '.') {
arg->prec = 0; arg->prec = 0;
if (--ctx->fmtcnt >= 0) { if (--ctx->fmtcnt >= 0) {
@ -13613,6 +13608,9 @@ unicode_format_arg_parse(struct unicode_formatter_t *ctx,
- "e", "E", "f", "F", "g", "G": float - "e", "E", "f", "F", "g", "G": float
- "c": int or str (1 character) - "c": int or str (1 character)
When possible, the output is written directly into the Unicode writer
(ctx->writer). A string is created when padding is required.
Return 0 if the argument has been formatted into *p_str, Return 0 if the argument has been formatted into *p_str,
1 if the argument has been written into ctx->writer, 1 if the argument has been written into ctx->writer,
-1 on error. */ -1 on error. */
@ -13639,10 +13637,8 @@ unicode_format_arg_format(struct unicode_formatter_t *ctx,
if (v == NULL) if (v == NULL)
return -1; return -1;
arg->sign = 0;
switch (arg->ch) { switch (arg->ch) {
case 's': case 's':
case 'r': case 'r':
case 'a': case 'a':
@ -13894,6 +13890,13 @@ unicode_format_arg(struct unicode_formatter_t *ctx)
PyObject *str; PyObject *str;
int ret; int ret;
arg.ch = PyUnicode_READ(ctx->fmtkind, ctx->fmtdata, ctx->fmtpos);
arg.flags = 0;
arg.width = -1;
arg.prec = -1;
arg.sign = 0;
str = NULL;
ret = unicode_format_arg_parse(ctx, &arg); ret = unicode_format_arg_parse(ctx, &arg);
if (ret == -1) if (ret == -1)
return -1; return -1;