diff options
| author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2018-02-01 17:41:24 (GMT) |
|---|---|---|
| committer | Christian Heimes <christian@python.org> | 2018-02-01 17:41:24 (GMT) |
| commit | 78758f29b13aba6136f4c0a15d4457fbf92c5eef (patch) | |
| tree | 407e106bb164a14fae8431c74f0b343eb1d9de9d /Python/ast_unparse.c | |
| parent | 4d6543005df6eba6ec3414686fdc251cb4fe9792 (diff) | |
| download | cpython-78758f29b13aba6136f4c0a15d4457fbf92c5eef.zip cpython-78758f29b13aba6136f4c0a15d4457fbf92c5eef.tar.gz cpython-78758f29b13aba6136f4c0a15d4457fbf92c5eef.tar.bz2 | |
[3.7] bpo-32711: Fix warnings for Python/ast_unparse.c (GH-5426) (#5475)
* bpo-32711: Fix warnings for Python/ast_unparse.c
(cherry picked from commit 83ab995871ffd504ac229bdbf5b9e9ffc1032815)
Co-authored-by: Stéphane Wirtel <stephane@wirtel.be>
Diffstat (limited to 'Python/ast_unparse.c')
| -rw-r--r-- | Python/ast_unparse.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/Python/ast_unparse.c b/Python/ast_unparse.c index ef9e948..1345271 100644 --- a/Python/ast_unparse.c +++ b/Python/ast_unparse.c @@ -20,7 +20,7 @@ append_formattedvalue(_PyUnicodeWriter *writer, expr_ty e, bool is_format_spec); static int append_charp(_PyUnicodeWriter *writer, const char *charp) { - return _PyUnicodeWriter_WriteASCIIString(writer, charp, -1); + return _PyUnicodeWriter_WriteASCIIString(writer, charp, -1); } static int @@ -100,6 +100,8 @@ append_ast_binop(_PyUnicodeWriter *writer, expr_ty e, bool omit_parens) case BitAnd: op = " & "; break; case FloorDiv: op = " // "; break; case Pow: op = " ** "; break; + default: + Py_UNREACHABLE(); } if (-1 == append_charp(writer, op)) { @@ -127,6 +129,8 @@ append_ast_unaryop(_PyUnicodeWriter *writer, expr_ty e, bool omit_parens) case Not: op = "not "; break; case UAdd: op = "+"; break; case USub: op = "-"; break; + default: + Py_UNREACHABLE(); } if (-1 == append_charp(writer, op)) { @@ -856,7 +860,7 @@ append_formattedvalue(_PyUnicodeWriter *writer, expr_ty e, bool is_format_spec) return -1; } } - if (e->v.FormattedValue.format_spec > 0) { + if (e->v.FormattedValue.format_spec) { if (-1 == _PyUnicodeWriter_WriteASCIIString(writer, ":", 1) || -1 == append_fstring_element(writer, e->v.FormattedValue.format_spec, @@ -1119,7 +1123,7 @@ append_ast_expr(_PyUnicodeWriter *writer, expr_ty e, bool omit_parens) } static int -maybe_init_static_strings() +maybe_init_static_strings(void) { if (!_str_open_br && !(_str_open_br = PyUnicode_InternFromString("{"))) { |