X-Git-Url: https://code.delx.au/gnu-emacs/blobdiff_plain/a7af7fdede602a111401c2352e81311a9dc38b99..fb39b937b0628f4592b07d0aa61a41cf696abd30:/src/casefiddle.c diff --git a/src/casefiddle.c b/src/casefiddle.c index 1a0a62f273..e365462757 100644 --- a/src/casefiddle.c +++ b/src/casefiddle.c @@ -1,6 +1,6 @@ /* GNU Emacs case conversion functions. -Copyright (C) 1985, 1994, 1997-1999, 2001-2011 Free Software Foundation, Inc. +Copyright (C) 1985, 1994, 1997-1999, 2001-2012 Free Software Foundation, Inc. This file is part of GNU Emacs. @@ -19,10 +19,10 @@ along with GNU Emacs. If not, see . */ #include -#include + #include "lisp.h" -#include "buffer.h" #include "character.h" +#include "buffer.h" #include "commands.h" #include "syntax.h" #include "composite.h" @@ -35,8 +35,8 @@ Lisp_Object Qidentity; static Lisp_Object casify_object (enum case_action flag, Lisp_Object obj) { - register int c, c1; - register int inword = flag == CASE_DOWN; + int c, c1; + bool inword = flag == CASE_DOWN; /* If the case table is flagged as modified, rescan it. */ if (NILP (XCHAR_TABLE (BVAR (current_buffer, downcase_table))->extras[1])) @@ -47,7 +47,8 @@ casify_object (enum case_action flag, Lisp_Object obj) int flagbits = (CHAR_ALT | CHAR_SUPER | CHAR_HYPER | CHAR_SHIFT | CHAR_CTL | CHAR_META); int flags = XINT (obj) & flagbits; - int multibyte = ! NILP (BVAR (current_buffer, enable_multibyte_characters)); + bool multibyte = ! NILP (BVAR (current_buffer, + enable_multibyte_characters)); /* If the character has higher bits set above the flags, return it unchanged. @@ -82,8 +83,8 @@ casify_object (enum case_action flag, Lisp_Object obj) wrong_type_argument (Qchar_or_string_p, obj); else if (!STRING_MULTIBYTE (obj)) { - EMACS_INT i; - EMACS_INT size = SCHARS (obj); + ptrdiff_t i; + ptrdiff_t size = SCHARS (obj); obj = Fcopy_sequence (obj); for (i = 0; i < size; i++) @@ -111,26 +112,19 @@ casify_object (enum case_action flag, Lisp_Object obj) } else { - EMACS_INT i, i_byte, size = SCHARS (obj); + ptrdiff_t i, i_byte, size = SCHARS (obj); int len; USE_SAFE_ALLOCA; - unsigned char *dst, *o; - /* Over-allocate by 12%: this is a minor overhead, but should be - sufficient in 99.999% of the cases to avoid a reallocation. */ - EMACS_INT o_size = SBYTES (obj) + SBYTES (obj) / 8 + MAX_MULTIBYTE_LENGTH; - SAFE_ALLOCA (dst, void *, o_size); - o = dst; + ptrdiff_t o_size = (size < STRING_BYTES_BOUND / MAX_MULTIBYTE_LENGTH + ? size * MAX_MULTIBYTE_LENGTH + : STRING_BYTES_BOUND); + unsigned char *dst = SAFE_ALLOCA (o_size); + unsigned char *o = dst; for (i = i_byte = 0; i < size; i++, i_byte += len) { - if ((o - dst) + MAX_MULTIBYTE_LENGTH > o_size) - { /* Not enough space for the next char: grow the destination. */ - unsigned char *old_dst = dst; - o_size += o_size; /* Probably overkill, but extremely rare. */ - SAFE_ALLOCA (dst, void *, o_size); - memcpy (dst, old_dst, o - old_dst); - o = dst + (o - old_dst); - } + if (o_size - (o - dst) < MAX_MULTIBYTE_LENGTH) + string_overflow (); c = STRING_CHAR_AND_LENGTH (SDATA (obj) + i_byte, len); if (inword && flag != CASE_CAPITALIZE_UP) c = downcase (c); @@ -196,17 +190,17 @@ The argument object is not altered--the value is a copy. */) static void casify_region (enum case_action flag, Lisp_Object b, Lisp_Object e) { - register int c; - register int inword = flag == CASE_DOWN; - register int multibyte = !NILP (BVAR (current_buffer, enable_multibyte_characters)); - EMACS_INT start, end; - EMACS_INT start_byte; + int c; + bool inword = flag == CASE_DOWN; + bool multibyte = !NILP (BVAR (current_buffer, enable_multibyte_characters)); + ptrdiff_t start, end; + ptrdiff_t start_byte; /* Position of first and last changes. */ - EMACS_INT first = -1, last IF_LINT (= 0); + ptrdiff_t first = -1, last IF_LINT (= 0); - EMACS_INT opoint = PT; - EMACS_INT opoint_byte = PT_BYTE; + ptrdiff_t opoint = PT; + ptrdiff_t opoint_byte = PT_BYTE; if (EQ (b, e)) /* Not modifying because nothing marked */ @@ -223,7 +217,7 @@ casify_region (enum case_action flag, Lisp_Object b, Lisp_Object e) record_change (start, end - start); start_byte = CHAR_TO_BYTE (start); - SETUP_BUFFER_SYNTAX_TABLE(); /* For syntax_prefix_flag_p. */ + SETUP_BUFFER_SYNTAX_TABLE (); /* For syntax_prefix_flag_p. */ while (start < end) { @@ -351,10 +345,10 @@ character positions to operate on. */) } static Lisp_Object -operate_on_word (Lisp_Object arg, EMACS_INT *newpoint) +operate_on_word (Lisp_Object arg, ptrdiff_t *newpoint) { Lisp_Object val; - EMACS_INT farend; + ptrdiff_t farend; EMACS_INT iarg; CHECK_NUMBER (arg); @@ -376,7 +370,7 @@ See also `capitalize-word'. */) (Lisp_Object arg) { Lisp_Object beg, end; - EMACS_INT newpoint; + ptrdiff_t newpoint; XSETFASTINT (beg, PT); end = operate_on_word (arg, &newpoint); casify_region (CASE_UP, beg, end); @@ -390,7 +384,7 @@ With negative argument, convert previous words but do not move. */) (Lisp_Object arg) { Lisp_Object beg, end; - EMACS_INT newpoint; + ptrdiff_t newpoint; XSETFASTINT (beg, PT); end = operate_on_word (arg, &newpoint); casify_region (CASE_DOWN, beg, end); @@ -406,7 +400,7 @@ With negative argument, capitalize previous words but do not move. */) (Lisp_Object arg) { Lisp_Object beg, end; - EMACS_INT newpoint; + ptrdiff_t newpoint; XSETFASTINT (beg, PT); end = operate_on_word (arg, &newpoint); casify_region (CASE_CAPITALIZE, beg, end); @@ -417,8 +411,7 @@ With negative argument, capitalize previous words but do not move. */) void syms_of_casefiddle (void) { - Qidentity = intern_c_string ("identity"); - staticpro (&Qidentity); + DEFSYM (Qidentity, "identity"); defsubr (&Supcase); defsubr (&Sdowncase); defsubr (&Scapitalize); @@ -435,9 +428,9 @@ syms_of_casefiddle (void) void keys_of_casefiddle (void) { - initial_define_key (control_x_map, Ctl('U'), "upcase-region"); + initial_define_key (control_x_map, Ctl ('U'), "upcase-region"); Fput (intern ("upcase-region"), Qdisabled, Qt); - initial_define_key (control_x_map, Ctl('L'), "downcase-region"); + initial_define_key (control_x_map, Ctl ('L'), "downcase-region"); Fput (intern ("downcase-region"), Qdisabled, Qt); initial_define_key (meta_map, 'u', "upcase-word");