diff options
| author | Andreas Schwab | 2011-01-30 01:35:35 +0100 |
|---|---|---|
| committer | Andreas Schwab | 2011-01-30 01:35:35 +0100 |
| commit | 4ec88040a1c1ff9a280d40cbcdc083b1121654de (patch) | |
| tree | d019d239443dd6aa2ccb9217ae4659305842a5af /src | |
| parent | a2b6e5d60b10c6baa3fbc36bfb158342c1c424ab (diff) | |
| download | emacs-4ec88040a1c1ff9a280d40cbcdc083b1121654de.tar.gz emacs-4ec88040a1c1ff9a280d40cbcdc083b1121654de.zip | |
* src/font.c (PROP_MATCH): Remove parameter N and use strlen instead.
All uses changed.
(PROP_SAVE): Likewise.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ChangeLog | 6 | ||||
| -rw-r--r-- | src/font.c | 78 |
2 files changed, 46 insertions, 38 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index 0c562f87994..50a7a217245 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -1,3 +1,9 @@ | |||
| 1 | 2011-01-30 Andreas Schwab <schwab@linux-m68k.org> | ||
| 2 | |||
| 3 | * font.c (PROP_MATCH): Remove parameter N and use strlen instead. | ||
| 4 | All uses changed. | ||
| 5 | (PROP_SAVE): Likewise. | ||
| 6 | |||
| 1 | 2011-01-29 Chong Yidong <cyd@stupidchicken.com> | 7 | 2011-01-29 Chong Yidong <cyd@stupidchicken.com> |
| 2 | 8 | ||
| 3 | * keyboard.c (make_lispy_position): Fix typo in last change | 9 | * keyboard.c (make_lispy_position): Fix typo in last change |
diff --git a/src/font.c b/src/font.c index 449169dad1b..58d8dc90962 100644 --- a/src/font.c +++ b/src/font.c | |||
| @@ -1380,25 +1380,26 @@ font_parse_fcname (char *name, Lisp_Object font) | |||
| 1380 | word_len = q - p; | 1380 | word_len = q - p; |
| 1381 | val = font_intern_prop (p, q - p, 1); | 1381 | val = font_intern_prop (p, q - p, 1); |
| 1382 | 1382 | ||
| 1383 | #define PROP_MATCH(STR,N) ((word_len == N) && memcmp (p, STR, N) == 0) | 1383 | #define PROP_MATCH(STR) (word_len == strlen (STR) \ |
| 1384 | 1384 | && memcmp (p, STR, strlen (STR)) == 0) | |
| 1385 | if (PROP_MATCH ("light", 5) | 1385 | |
| 1386 | || PROP_MATCH ("medium", 6) | 1386 | if (PROP_MATCH ("light") |
| 1387 | || PROP_MATCH ("demibold", 8) | 1387 | || PROP_MATCH ("medium") |
| 1388 | || PROP_MATCH ("bold", 4) | 1388 | || PROP_MATCH ("demibold") |
| 1389 | || PROP_MATCH ("black", 5)) | 1389 | || PROP_MATCH ("bold") |
| 1390 | || PROP_MATCH ("black")) | ||
| 1390 | FONT_SET_STYLE (font, FONT_WEIGHT_INDEX, val); | 1391 | FONT_SET_STYLE (font, FONT_WEIGHT_INDEX, val); |
| 1391 | else if (PROP_MATCH ("roman", 5) | 1392 | else if (PROP_MATCH ("roman") |
| 1392 | || PROP_MATCH ("italic", 6) | 1393 | || PROP_MATCH ("italic") |
| 1393 | || PROP_MATCH ("oblique", 7)) | 1394 | || PROP_MATCH ("oblique")) |
| 1394 | FONT_SET_STYLE (font, FONT_SLANT_INDEX, val); | 1395 | FONT_SET_STYLE (font, FONT_SLANT_INDEX, val); |
| 1395 | else if (PROP_MATCH ("charcell", 8)) | 1396 | else if (PROP_MATCH ("charcell")) |
| 1396 | ASET (font, FONT_SPACING_INDEX, | 1397 | ASET (font, FONT_SPACING_INDEX, |
| 1397 | make_number (FONT_SPACING_CHARCELL)); | 1398 | make_number (FONT_SPACING_CHARCELL)); |
| 1398 | else if (PROP_MATCH ("mono", 4)) | 1399 | else if (PROP_MATCH ("mono")) |
| 1399 | ASET (font, FONT_SPACING_INDEX, | 1400 | ASET (font, FONT_SPACING_INDEX, |
| 1400 | make_number (FONT_SPACING_MONO)); | 1401 | make_number (FONT_SPACING_MONO)); |
| 1401 | else if (PROP_MATCH ("proportional", 12)) | 1402 | else if (PROP_MATCH ("proportional")) |
| 1402 | ASET (font, FONT_SPACING_INDEX, | 1403 | ASET (font, FONT_SPACING_INDEX, |
| 1403 | make_number (FONT_SPACING_PROPORTIONAL)); | 1404 | make_number (FONT_SPACING_PROPORTIONAL)); |
| 1404 | #undef PROP_MATCH | 1405 | #undef PROP_MATCH |
| @@ -1480,31 +1481,32 @@ font_parse_fcname (char *name, Lisp_Object font) | |||
| 1480 | word_start = q + 1; | 1481 | word_start = q + 1; |
| 1481 | word_len = p - word_start; | 1482 | word_len = p - word_start; |
| 1482 | 1483 | ||
| 1483 | #define PROP_MATCH(STR,N) \ | 1484 | #define PROP_MATCH(STR) \ |
| 1484 | ((word_len == N) && memcmp (word_start, STR, N) == 0) | 1485 | (word_len == strlen (STR) \ |
| 1485 | #define PROP_SAVE(VAR,STR,N) \ | 1486 | && memcmp (word_start, STR, strlen (STR)) == 0) |
| 1486 | (VAR = NILP (VAR) ? font_intern_prop (STR, N, 1) : VAR) | 1487 | #define PROP_SAVE(VAR, STR) \ |
| 1487 | 1488 | (VAR = NILP (VAR) ? font_intern_prop (STR, strlen (STR), 1) : VAR) | |
| 1488 | if (PROP_MATCH ("Ultra-Light", 11)) | 1489 | |
| 1489 | PROP_SAVE (weight, "ultra-light", 11); | 1490 | if (PROP_MATCH ("Ultra-Light")) |
| 1490 | else if (PROP_MATCH ("Light", 5)) | 1491 | PROP_SAVE (weight, "ultra-light"); |
| 1491 | PROP_SAVE (weight, "light", 5); | 1492 | else if (PROP_MATCH ("Light")) |
| 1492 | else if (PROP_MATCH ("Book", 4)) | 1493 | PROP_SAVE (weight, "light"); |
| 1493 | PROP_SAVE (weight, "book", 4); | 1494 | else if (PROP_MATCH ("Book")) |
| 1494 | else if (PROP_MATCH ("Medium", 6)) | 1495 | PROP_SAVE (weight, "book"); |
| 1495 | PROP_SAVE (weight, "medium", 6); | 1496 | else if (PROP_MATCH ("Medium")) |
| 1496 | else if (PROP_MATCH ("Semi-Bold", 9)) | 1497 | PROP_SAVE (weight, "medium"); |
| 1497 | PROP_SAVE (weight, "semi-bold", 9); | 1498 | else if (PROP_MATCH ("Semi-Bold")) |
| 1498 | else if (PROP_MATCH ("Bold", 4)) | 1499 | PROP_SAVE (weight, "semi-bold"); |
| 1499 | PROP_SAVE (weight, "bold", 4); | 1500 | else if (PROP_MATCH ("Bold")) |
| 1500 | else if (PROP_MATCH ("Italic", 6)) | 1501 | PROP_SAVE (weight, "bold"); |
| 1501 | PROP_SAVE (slant, "italic", 6); | 1502 | else if (PROP_MATCH ("Italic")) |
| 1502 | else if (PROP_MATCH ("Oblique", 7)) | 1503 | PROP_SAVE (slant, "italic"); |
| 1503 | PROP_SAVE (slant, "oblique", 7); | 1504 | else if (PROP_MATCH ("Oblique")) |
| 1504 | else if (PROP_MATCH ("Semi-Condensed", 14)) | 1505 | PROP_SAVE (slant, "oblique"); |
| 1505 | PROP_SAVE (width, "semi-condensed", 14); | 1506 | else if (PROP_MATCH ("Semi-Condensed")) |
| 1506 | else if (PROP_MATCH ("Condensed", 9)) | 1507 | PROP_SAVE (width, "semi-condensed"); |
| 1507 | PROP_SAVE (width, "condensed", 9); | 1508 | else if (PROP_MATCH ("Condensed")) |
| 1509 | PROP_SAVE (width, "condensed"); | ||
| 1508 | /* An unknown word must be part of the font name. */ | 1510 | /* An unknown word must be part of the font name. */ |
| 1509 | else | 1511 | else |
| 1510 | { | 1512 | { |