diff options
| author | Eli Zaretskii | 2011-04-23 13:33:28 +0300 |
|---|---|---|
| committer | Eli Zaretskii | 2011-04-23 13:33:28 +0300 |
| commit | e6c3da2065ac72cc4e1a2bef22d367cd75401892 (patch) | |
| tree | 1f6fcbee1e12f63096d2221a89f5436b831862a9 /src/coding.c | |
| parent | 4ffd0d6b569d252e4e807d4e9c9d6a5bd5b08640 (diff) | |
| download | emacs-e6c3da2065ac72cc4e1a2bef22d367cd75401892.tar.gz emacs-e6c3da2065ac72cc4e1a2bef22d367cd75401892.zip | |
Fix doprnt so it could be used safely in `verror'. (Bug#8435)
src/doprnt.c: Include limits.h.
(SIZE_MAX): New macro.
(doprnt): Return a size_t value. 2nd arg is now size_t. Many
local variables are now size_t instead of int or unsigned.
Improve overflow protection. Support `l' modifier for integer
conversions. Support %l conversion. Don't assume an EMACS_INT
argument for integer conversions and for %c.
src/lisp.h (doprnt): Restore prototype.
src/makefile.w32-in ($(BLD)/callint.$(O)): Depend on
$(SRC)/character.h.
src/Makefile.in (base_obj): Add back doprnt.o.
src/deps.mk (doprnt.o): Add back prerequisites.
(callint.o): Depend on character.h.
src/eval.c (internal_lisp_condition_case): Include the handler
representation in the error message.
(verror): Call doprnt instead of vsnprintf. Fix an off-by-one bug
when breaking from the loop.
src/xdisp.c (vmessage): Call doprnt instead of vsnprintf.
src/callint.c (Fcall_interactively): When displaying error message
about invalid control letter, pass the character's codepoint, not
a pointer to its multibyte form. Improve display of the character
in octal and display also its hex code.
src/character.c (char_string): Use %x to display the (unsigned)
codepoint of an invalid character, to avoid displaying a bogus
negative value.
src/font.c (check_otf_features): Pass SDATA of SYMBOL_NAME to
`error', not SYMBOL_NAME itself.
src/coding.c (Fencode_sjis_char, Fencode_big5_char): Use %c for
character arguments to `error'.
src/charset.c (check_iso_charset_parameter): Fix incorrect argument
to `error' in error message about FINAL_CHAR argument. Make sure
FINAL_CHAR is a character, and use %c when it is passed as
argument to `error'.
Diffstat (limited to 'src/coding.c')
| -rw-r--r-- | src/coding.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/coding.c b/src/coding.c index b49070e5e16..221ada51158 100644 --- a/src/coding.c +++ b/src/coding.c | |||
| @@ -9071,7 +9071,7 @@ Return the corresponding code in SJIS. */) | |||
| 9071 | charset_list = CODING_ATTR_CHARSET_LIST (attrs); | 9071 | charset_list = CODING_ATTR_CHARSET_LIST (attrs); |
| 9072 | charset = char_charset (c, charset_list, &code); | 9072 | charset = char_charset (c, charset_list, &code); |
| 9073 | if (code == CHARSET_INVALID_CODE (charset)) | 9073 | if (code == CHARSET_INVALID_CODE (charset)) |
| 9074 | error ("Can't encode by shift_jis encoding: %d", c); | 9074 | error ("Can't encode by shift_jis encoding: %c", c); |
| 9075 | JIS_TO_SJIS (code); | 9075 | JIS_TO_SJIS (code); |
| 9076 | 9076 | ||
| 9077 | return make_number (code); | 9077 | return make_number (code); |
| @@ -9142,7 +9142,7 @@ Return the corresponding character code in Big5. */) | |||
| 9142 | charset_list = CODING_ATTR_CHARSET_LIST (attrs); | 9142 | charset_list = CODING_ATTR_CHARSET_LIST (attrs); |
| 9143 | charset = char_charset (c, charset_list, &code); | 9143 | charset = char_charset (c, charset_list, &code); |
| 9144 | if (code == CHARSET_INVALID_CODE (charset)) | 9144 | if (code == CHARSET_INVALID_CODE (charset)) |
| 9145 | error ("Can't encode by Big5 encoding: %d", c); | 9145 | error ("Can't encode by Big5 encoding: %c", c); |
| 9146 | 9146 | ||
| 9147 | return make_number (code); | 9147 | return make_number (code); |
| 9148 | } | 9148 | } |