diff options
| author | Guido van Rossum <guido@python.org> | 1998-01-19 22:07:46 +0000 |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 1998-01-19 22:07:46 +0000 |
| commit | f6970b93407015cb10d473f48045bd368bc28bda (patch) | |
| tree | f8dc775b78e6b26bc0cb335c4a6e74291feb6ca2 /Parser | |
| parent | fedf086c6c2450eb73f72ebe42c02a6d13b3fdd0 (diff) | |
| download | cpython-f6970b93407015cb10d473f48045bd368bc28bda.tar.gz | |
tok_nextc() should return unsigned characters, to avoid mistaking
'\377' for EOF.
Diffstat (limited to 'Parser')
| -rw-r--r-- | Parser/tokenizer.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/Parser/tokenizer.c b/Parser/tokenizer.c index 3dc6c82799..134f00f252 100644 --- a/Parser/tokenizer.c +++ b/Parser/tokenizer.c @@ -46,6 +46,14 @@ extern char *PyOS_Readline Py_PROTO((char *)); /* Don't ever change this -- it would break the portability of Python code */ #define TABSIZE 8 +/* Convert a possibly signed character to a nonnegative int */ +/* XXX This assumes characters are 8 bits wide */ +#ifdef __CHAR_UNSIGNED__ +#define Py_CHARMASK(c) (c) +#else +#define Py_CHARMASK(c) ((c) & 0xff) +#endif + /* Forward */ static struct tok_state *tok_new Py_PROTO((void)); static int tok_nextc Py_PROTO((struct tok_state *tok)); @@ -178,7 +186,7 @@ tok_nextc(tok) { for (;;) { if (tok->cur != tok->inp) { - return *tok->cur++; /* Fast path */ + return Py_CHARMASK(*tok->cur++); /* Fast path */ } if (tok->done != E_OK) return EOF; @@ -197,7 +205,7 @@ tok_nextc(tok) tok->buf = tok->cur; tok->lineno++; tok->inp = end; - return *tok->cur++; + return Py_CHARMASK(*tok->cur++); } if (tok->prompt != NULL) { char *new = PyOS_Readline(tok->prompt); |