]> git.eshelyaron.com Git - emacs.git/commitdiff
More casefiddle minor fixes
authorPaul Eggert <eggert@cs.ucla.edu>
Fri, 7 Apr 2017 03:02:15 +0000 (20:02 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Fri, 7 Apr 2017 03:03:21 +0000 (20:03 -0700)
* src/casefiddle.c (case_character_impl): Omit unnecessary casts.
(case_character_impl): Avoid reevaluation of CHAR_TABLE_REF.
(GREEK_CAPITAL_LETTER_SIGMA): Fix typo in my previous change.

src/casefiddle.c

index b7782a83c3d7f78c64fbfe4322eeaf5a5996692c..443d62b625932da941ef4e174aae75ba6bc8e6e4 100644 (file)
@@ -125,9 +125,9 @@ case_character_impl (struct casing_str_buf *buf,
     }
 
   /* Look through the special casing entries.  */
-  if (buf && !NILP (ctx->specialcase_char_tables[(int)flag]))
+  if (buf && !NILP (ctx->specialcase_char_tables[flag]))
     {
-      prop = CHAR_TABLE_REF (ctx->specialcase_char_tables[(int)flag], ch);
+      prop = CHAR_TABLE_REF (ctx->specialcase_char_tables[flag], ch);
       if (STRINGP (prop))
         {
           struct Lisp_String *str = XSTRING (prop);
@@ -144,12 +144,21 @@ case_character_impl (struct casing_str_buf *buf,
   /* Handle simple, one-to-one case.  */
   if (flag == CASE_DOWN)
     cased = downcase (ch);
-  else if (!NILP (ctx->titlecase_char_table)
-          && CHARACTERP (prop
-                         = CHAR_TABLE_REF (ctx->titlecase_char_table, ch)))
-    cased = XFASTINT (prop);
   else
-    cased = upcase (ch);
+    {
+      bool cased_is_set = false;
+      if (!NILP (ctx->titlecase_char_table))
+       {
+         prop = CHAR_TABLE_REF (ctx->titlecase_char_table, ch);
+         if (CHARACTERP (prop))
+           {
+             cased = XFASTINT (prop);
+             cased_is_set = true;
+           }
+       }
+      if (!cased_is_set)
+       cased = upcase (ch);
+    }
 
   /* And we’re done.  */
  done:
@@ -167,7 +176,7 @@ case_character_impl (struct casing_str_buf *buf,
    The rule does not conflict with any other casing rules so while it is
    a conditional one, it is independent of language.  */
 
-enum { GREEK_CAPITAL_LETTER_SIGMA = 0x03A }; /* Σ */
+enum { GREEK_CAPITAL_LETTER_SIGMA = 0x03A3 }; /* Σ */
 enum { GREEK_SMALL_LETTER_FINAL_SIGMA = 0x03C2 }; /* ς */
 \f
 /* Based on CTX, case character CH accordingly.  Update CTX as necessary.