diff options
| author | Paul Eggert | 2011-03-16 01:34:44 -0700 |
|---|---|---|
| committer | Paul Eggert | 2011-03-16 01:34:44 -0700 |
| commit | 13a547c6792935558a306bec264e0bad575cec87 (patch) | |
| tree | a1c6208a30ad452837bd81df1f3ab66a73907798 /src | |
| parent | 170a2692ec2abdc4e4978206ddec6bebb6c989b2 (diff) | |
| download | emacs-13a547c6792935558a306bec264e0bad575cec87.tar.gz emacs-13a547c6792935558a306bec264e0bad575cec87.zip | |
* font.c: (font_style_to_value, font_prop_validate_style, font_unparse_fcname):
(font_update_drivers, Ffont_get_glyphs, font_add_log):
Rename or move locals to avoid shadowing.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ChangeLog | 3 | ||||
| -rw-r--r-- | src/font.c | 30 |
2 files changed, 18 insertions, 15 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index 4b3f0ceca36..b056c08419e 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -7,6 +7,9 @@ | |||
| 7 | (font_parse_fcname): Remove unused vars. | 7 | (font_parse_fcname): Remove unused vars. |
| 8 | (font_delete_unmatched): Now static. | 8 | (font_delete_unmatched): Now static. |
| 9 | (font_get_spec): Remove; unused. | 9 | (font_get_spec): Remove; unused. |
| 10 | (font_style_to_value, font_prop_validate_style, font_unparse_fcname): | ||
| 11 | (font_update_drivers, Ffont_get_glyphs, font_add_log): | ||
| 12 | Rename or move locals to avoid shadowing. | ||
| 10 | 13 | ||
| 11 | * fns.c (require_nesting_list, require_unwind): Now static. | 14 | * fns.c (require_nesting_list, require_unwind): Now static. |
| 12 | (Ffillarray): Rename locals to avoid shadowing. | 15 | (Ffillarray): Rename locals to avoid shadowing. |
diff --git a/src/font.c b/src/font.c index ed3b3aeefc5..a9a381a828d 100644 --- a/src/font.c +++ b/src/font.c | |||
| @@ -311,10 +311,10 @@ font_style_to_value (enum font_property_index prop, Lisp_Object val, int noerror | |||
| 311 | { | 311 | { |
| 312 | Lisp_Object table = AREF (font_style_table, prop - FONT_WEIGHT_INDEX); | 312 | Lisp_Object table = AREF (font_style_table, prop - FONT_WEIGHT_INDEX); |
| 313 | int len = ASIZE (table); | 313 | int len = ASIZE (table); |
| 314 | int i, j; | ||
| 315 | 314 | ||
| 316 | if (SYMBOLP (val)) | 315 | if (SYMBOLP (val)) |
| 317 | { | 316 | { |
| 317 | int i, j; | ||
| 318 | char *s; | 318 | char *s; |
| 319 | Lisp_Object args[2], elt; | 319 | Lisp_Object args[2], elt; |
| 320 | 320 | ||
| @@ -500,10 +500,9 @@ font_prop_validate_style (Lisp_Object style, Lisp_Object val) | |||
| 500 | enum font_property_index prop = (EQ (style, QCweight) ? FONT_WEIGHT_INDEX | 500 | enum font_property_index prop = (EQ (style, QCweight) ? FONT_WEIGHT_INDEX |
| 501 | : EQ (style, QCslant) ? FONT_SLANT_INDEX | 501 | : EQ (style, QCslant) ? FONT_SLANT_INDEX |
| 502 | : FONT_WIDTH_INDEX); | 502 | : FONT_WIDTH_INDEX); |
| 503 | int n; | ||
| 504 | if (INTEGERP (val)) | 503 | if (INTEGERP (val)) |
| 505 | { | 504 | { |
| 506 | n = XINT (val); | 505 | int n = XINT (val); |
| 507 | if (((n >> 4) & 0xF) | 506 | if (((n >> 4) & 0xF) |
| 508 | >= ASIZE (AREF (font_style_table, prop - FONT_WEIGHT_INDEX))) | 507 | >= ASIZE (AREF (font_style_table, prop - FONT_WEIGHT_INDEX))) |
| 509 | val = Qerror; | 508 | val = Qerror; |
| @@ -1602,15 +1601,15 @@ font_unparse_fcname (Lisp_Object font, int pixel_size, char *name, int nbytes) | |||
| 1602 | len += strlen (":scalable=false"); /* or ":scalable=true" */ | 1601 | len += strlen (":scalable=false"); /* or ":scalable=true" */ |
| 1603 | for (tail = AREF (font, FONT_EXTRA_INDEX); CONSP (tail); tail = XCDR (tail)) | 1602 | for (tail = AREF (font, FONT_EXTRA_INDEX); CONSP (tail); tail = XCDR (tail)) |
| 1604 | { | 1603 | { |
| 1605 | Lisp_Object key = XCAR (XCAR (tail)), val = XCDR (XCAR (tail)); | 1604 | Lisp_Object key = XCAR (XCAR (tail)), value = XCDR (XCAR (tail)); |
| 1606 | 1605 | ||
| 1607 | len += SBYTES (SYMBOL_NAME (key)) + 1; /* for :KEY= */ | 1606 | len += SBYTES (SYMBOL_NAME (key)) + 1; /* for :KEY= */ |
| 1608 | if (STRINGP (val)) | 1607 | if (STRINGP (value)) |
| 1609 | len += SBYTES (val); | 1608 | len += SBYTES (value); |
| 1610 | else if (INTEGERP (val)) | 1609 | else if (INTEGERP (value)) |
| 1611 | len += sprintf (work, "%ld", (long) XINT (val)); | 1610 | len += sprintf (work, "%ld", (long) XINT (value)); |
| 1612 | else if (SYMBOLP (val)) | 1611 | else if (SYMBOLP (value)) |
| 1613 | len += (NILP (val) ? 5 : 4); /* for "false" or "true" */ | 1612 | len += (NILP (value) ? 5 : 4); /* for "false" or "true" */ |
| 1614 | } | 1613 | } |
| 1615 | 1614 | ||
| 1616 | if (len > nbytes) | 1615 | if (len > nbytes) |
| @@ -3418,14 +3417,13 @@ Lisp_Object | |||
| 3418 | font_update_drivers (FRAME_PTR f, Lisp_Object new_drivers) | 3417 | font_update_drivers (FRAME_PTR f, Lisp_Object new_drivers) |
| 3419 | { | 3418 | { |
| 3420 | Lisp_Object active_drivers = Qnil; | 3419 | Lisp_Object active_drivers = Qnil; |
| 3421 | struct font_driver *driver; | ||
| 3422 | struct font_driver_list *list; | 3420 | struct font_driver_list *list; |
| 3423 | 3421 | ||
| 3424 | /* At first, turn off non-requested drivers, and turn on requested | 3422 | /* At first, turn off non-requested drivers, and turn on requested |
| 3425 | drivers. */ | 3423 | drivers. */ |
| 3426 | for (list = f->font_driver_list; list; list = list->next) | 3424 | for (list = f->font_driver_list; list; list = list->next) |
| 3427 | { | 3425 | { |
| 3428 | driver = list->driver; | 3426 | struct font_driver *driver = list->driver; |
| 3429 | if ((EQ (new_drivers, Qt) || ! NILP (Fmemq (driver->type, new_drivers))) | 3427 | if ((EQ (new_drivers, Qt) || ! NILP (Fmemq (driver->type, new_drivers))) |
| 3430 | != list->on) | 3428 | != list->on) |
| 3431 | { | 3429 | { |
| @@ -4651,7 +4649,7 @@ the corresponding element is nil. */) | |||
| 4651 | Lisp_Object object) | 4649 | Lisp_Object object) |
| 4652 | { | 4650 | { |
| 4653 | struct font *font; | 4651 | struct font *font; |
| 4654 | int i, len, c; | 4652 | int i, len; |
| 4655 | Lisp_Object *chars, vec; | 4653 | Lisp_Object *chars, vec; |
| 4656 | USE_SAFE_ALLOCA; | 4654 | USE_SAFE_ALLOCA; |
| 4657 | 4655 | ||
| @@ -4669,6 +4667,7 @@ the corresponding element is nil. */) | |||
| 4669 | bytepos = CHAR_TO_BYTE (charpos); | 4667 | bytepos = CHAR_TO_BYTE (charpos); |
| 4670 | for (i = 0; charpos < XFASTINT (to); i++) | 4668 | for (i = 0; charpos < XFASTINT (to); i++) |
| 4671 | { | 4669 | { |
| 4670 | int c; | ||
| 4672 | FETCH_CHAR_ADVANCE (c, charpos, bytepos); | 4671 | FETCH_CHAR_ADVANCE (c, charpos, bytepos); |
| 4673 | chars[i] = make_number (c); | 4672 | chars[i] = make_number (c); |
| 4674 | } | 4673 | } |
| @@ -4690,7 +4689,7 @@ the corresponding element is nil. */) | |||
| 4690 | if (STRING_MULTIBYTE (object)) | 4689 | if (STRING_MULTIBYTE (object)) |
| 4691 | for (i = 0; i < len; i++) | 4690 | for (i = 0; i < len; i++) |
| 4692 | { | 4691 | { |
| 4693 | c = STRING_CHAR_ADVANCE (p); | 4692 | int c = STRING_CHAR_ADVANCE (p); |
| 4694 | chars[i] = make_number (c); | 4693 | chars[i] = make_number (c); |
| 4695 | } | 4694 | } |
| 4696 | else | 4695 | else |
| @@ -4941,7 +4940,7 @@ static Lisp_Object Vfont_log_deferred; | |||
| 4941 | void | 4940 | void |
| 4942 | font_add_log (const char *action, Lisp_Object arg, Lisp_Object result) | 4941 | font_add_log (const char *action, Lisp_Object arg, Lisp_Object result) |
| 4943 | { | 4942 | { |
| 4944 | Lisp_Object tail, val; | 4943 | Lisp_Object val; |
| 4945 | int i; | 4944 | int i; |
| 4946 | 4945 | ||
| 4947 | if (EQ (Vfont_log, Qt)) | 4946 | if (EQ (Vfont_log, Qt)) |
| @@ -4997,6 +4996,7 @@ font_add_log (const char *action, Lisp_Object arg, Lisp_Object result) | |||
| 4997 | } | 4996 | } |
| 4998 | else if (CONSP (result)) | 4997 | else if (CONSP (result)) |
| 4999 | { | 4998 | { |
| 4999 | Lisp_Object tail; | ||
| 5000 | result = Fcopy_sequence (result); | 5000 | result = Fcopy_sequence (result); |
| 5001 | for (tail = result; CONSP (tail); tail = XCDR (tail)) | 5001 | for (tail = result; CONSP (tail); tail = XCDR (tail)) |
| 5002 | { | 5002 | { |