]> git.eshelyaron.com Git - emacs.git/commit
Fix mishandling of symbols that look like numbers
authorPaul Eggert <eggert@cs.ucla.edu>
Thu, 11 Oct 2018 06:17:18 +0000 (23:17 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Thu, 11 Oct 2018 06:19:42 +0000 (23:19 -0700)
commit5bd8cfc14d4b0c78c07e65a583f42a10c4cbc06d
treec51b2a470f62679f5e5c3f6f55d90aa0082d0133
parentfd3a48fcd8bb212ec12b9b10a79de0ae605ee93b
Fix mishandling of symbols that look like numbers

* src/bignum.c (make_neg_biguint): New function.
* src/lread.c (read1): Do not mishandle an unquoted symbol
with name equal to something like "1\0x", i.e., a string
of numeric form followed by a NUL byte.
Formerly these symbols were misread as numbers.
(string_to_number): Change last argument from an integer flag
to a pointer to the length.  This lets the caller figure out
how much of the prefix was used.  All callers changed.
Add a fast path if the integer (sans sign) fits in uintmax_t.
Update comments and simplify now that bignums are present.
* src/print.c (print_object): Fix quoting of symbols that look
like numbers, by relying on string_to_number for the tricky
cases rather than trying to redo its logic, incorrectly.  For
example, (read (prin1-to-string '\1e+NaN)) formerly returned
"1e+NaN", which was wrong: a backslash is needed in the output
to prevent it from being read as a NaN.  Escape NO_BREAK_SPACE
too, since lread.c treats it like SPACE.
* test/src/print-tests.el (print-read-roundtrip):
Add tests illustrating the abovementioned bugs.
src/bignum.c
src/data.c
src/lisp.h
src/lread.c
src/print.c
src/process.c
test/src/print-tests.el