* character.h (CHAR_VALID_P): Remove unused parameter.

* fontset.c, lisp.h, xdisp.c: All uses changed.

diff --git a/src/ChangeLog b/src/ChangeLog
index 22e64fb409ac3356df1af5d56c1c7783bdda2b9c..f26d50f6ce9d7735bbb80a8d2627ef211ba59af3 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,8 @@
 2011-06-13  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * character.h (CHAR_VALID_P): Remove unused parameter.
+       * fontset.c, lisp.h, xdisp.c: All uses changed.
+
        * editfns.c (Ftranslate_region_internal): Omit redundant test.
 
        * fns.c (concat): Minor tuning based on overflow analysis.

diff --git a/src/character.h b/src/character.h
index de97754cfc7e3895432373eef04be8be7ae1d11c..a447ad40ac3820edc58cc38238c1479c4f2af9eb 100644 (file)
--- a/src/character.h
+++ b/src/character.h
@@ -109,8 +109,8 @@ along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.  */
 /* Nonzero iff X is a character.  */
 #define CHARACTERP(x) (NATNUMP (x) && XFASTINT (x) <= MAX_CHAR)
 
-/* Nonzero iff C is valid as a character code.  GENERICP is not used.  */
-#define CHAR_VALID_P(c, genericp) UNSIGNED_CMP (c, <=, MAX_CHAR)
+/* Nonzero iff C is valid as a character code.  */
+#define CHAR_VALID_P(c) UNSIGNED_CMP (c, <=, MAX_CHAR)
 
 /* Check if Lisp object X is a character or not.  */
 #define CHECK_CHARACTER(x) \

diff --git a/src/fontset.c b/src/fontset.c
index fec3c56b03675f4e666a309e24c73177dbf441f0..3091f43d6e93a3a00d81f599ce976f9bfb7c60d1 100644 (file)
--- a/src/fontset.c
+++ b/src/fontset.c
@@ -1851,7 +1851,7 @@ DEFUN ("internal-char-font", Finternal_char_font, Sinternal_char_font, 1, 2, 0,
       face_id = face_at_buffer_position (w, pos, -1, -1, &dummy,
                                         pos + 100, 0, -1);
     }
-  if (! CHAR_VALID_P (c, 0))
+  if (! CHAR_VALID_P (c))
     return Qnil;
   face_id = FACE_FOR_CHAR (f, FACE_FROM_ID (f, face_id), c, pos, Qnil);
   face = FACE_FROM_ID (f, face_id);

diff --git a/src/lisp.h b/src/lisp.h
index 4dfed319490afd9591f431c814e9072711734afe..1324440c7000a8409c60257b3d008a5908dd0af6 100644 (file)
--- a/src/lisp.h
+++ b/src/lisp.h
@@ -1609,7 +1609,7 @@ typedef struct {
 #define SET_GLYPH(glyph, char, face) ((glyph).ch = (char), (glyph).face_id = (face))
 
 /* Return 1 if GLYPH contains valid character code.  */
-#define GLYPH_CHAR_VALID_P(glyph) CHAR_VALID_P (GLYPH_CHAR (glyph), 1)
+#define GLYPH_CHAR_VALID_P(glyph) CHAR_VALID_P (GLYPH_CHAR (glyph))
 
 
 /* Glyph Code from a display vector may either be an integer which
@@ -1623,7 +1623,7 @@ typedef struct {
   (CONSP (gc) ? XINT (XCDR (gc)) : INTEGERP (gc) ? (XINT (gc) >> CHARACTERBITS) : DEFAULT_FACE_ID)
 
 /* Return 1 if glyph code from display vector contains valid character code.  */
-#define GLYPH_CODE_CHAR_VALID_P(gc) CHAR_VALID_P (GLYPH_CODE_CHAR (gc), 1)
+#define GLYPH_CODE_CHAR_VALID_P(gc) CHAR_VALID_P (GLYPH_CODE_CHAR (gc))
 
 #define GLYPH_CODE_P(gc) ((CONSP (gc) && INTEGERP (XCAR (gc)) && INTEGERP (XCDR (gc))) || INTEGERP (gc))
 

diff --git a/src/xdisp.c b/src/xdisp.c
index c49565992950bb96b3a1ec0ef05a23036380c42b..b70e0dc0d0b44654ec65b93f33af9760df9ead20 100644 (file)
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -1335,7 +1335,7 @@ string_char_and_length (const unsigned char *str, int *len)
   int c;
 
   c = STRING_CHAR_AND_LENGTH (str, *len);
-  if (!CHAR_VALID_P (c, 1))
+  if (!CHAR_VALID_P (c))
     /* We may not change the length here because other places in Emacs
        don't use this function, i.e. they silently accept invalid
        characters.  */