diff options
| author | Paul Eggert | 2018-08-25 14:20:02 -0700 |
|---|---|---|
| committer | Paul Eggert | 2018-08-25 14:20:27 -0700 |
| commit | 2daf8b7e55698cc58f9b0a82aac591f957041cc6 (patch) | |
| tree | d806c9974de7de06653a0557aeb0c3ace682ea7b /src | |
| parent | ccdb08ef4ed8f96e79aa06cf5e806c9c487d58ad (diff) | |
| download | emacs-2daf8b7e55698cc58f9b0a82aac591f957041cc6.tar.gz emacs-2daf8b7e55698cc58f9b0a82aac591f957041cc6.zip | |
Prefer CONSP etc. to XTYPE
* src/nsmenu.m (process_dialog:):
(initFromContents:isQuestion:):
Prefer CONSP (x) to XTYPE (x) == Lisp_Cons,
and similarly for STRINGP (x).
Diffstat (limited to 'src')
| -rw-r--r-- | src/nsmenu.m | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/nsmenu.m b/src/nsmenu.m index 7010b773e3c..983e77763b9 100644 --- a/src/nsmenu.m +++ b/src/nsmenu.m | |||
| @@ -1632,24 +1632,24 @@ ns_popup_dialog (struct frame *f, Lisp_Object header, Lisp_Object contents) | |||
| 1632 | int row = 0; | 1632 | int row = 0; |
| 1633 | int buttons = 0, btnnr = 0; | 1633 | int buttons = 0, btnnr = 0; |
| 1634 | 1634 | ||
| 1635 | for (; XTYPE (lst) == Lisp_Cons; lst = XCDR (lst)) | 1635 | for (; CONSP (lst); lst = XCDR (lst)) |
| 1636 | { | 1636 | { |
| 1637 | item = XCAR (list); | 1637 | item = XCAR (list); |
| 1638 | if (XTYPE (item) == Lisp_Cons) | 1638 | if (CONSP (item)) |
| 1639 | ++buttons; | 1639 | ++buttons; |
| 1640 | } | 1640 | } |
| 1641 | 1641 | ||
| 1642 | if (buttons > 0) | 1642 | if (buttons > 0) |
| 1643 | button_values = xmalloc (buttons * sizeof *button_values); | 1643 | button_values = xmalloc (buttons * sizeof *button_values); |
| 1644 | 1644 | ||
| 1645 | for (; XTYPE (list) == Lisp_Cons; list = XCDR (list)) | 1645 | for (; CONSP (list); list = XCDR (list)) |
| 1646 | { | 1646 | { |
| 1647 | item = XCAR (list); | 1647 | item = XCAR (list); |
| 1648 | if (XTYPE (item) == Lisp_String) | 1648 | if (STRINGP (item)) |
| 1649 | { | 1649 | { |
| 1650 | [self addString: SSDATA (item) row: row++]; | 1650 | [self addString: SSDATA (item) row: row++]; |
| 1651 | } | 1651 | } |
| 1652 | else if (XTYPE (item) == Lisp_Cons) | 1652 | else if (CONSP (item)) |
| 1653 | { | 1653 | { |
| 1654 | button_values[btnnr] = XCDR (item); | 1654 | button_values[btnnr] = XCDR (item); |
| 1655 | [self addButton: SSDATA (XCAR (item)) value: btnnr row: row++]; | 1655 | [self addButton: SSDATA (XCAR (item)) value: btnnr row: row++]; |
| @@ -1726,7 +1726,7 @@ ns_popup_dialog (struct frame *f, Lisp_Object header, Lisp_Object contents) | |||
| 1726 | Lisp_Object head; | 1726 | Lisp_Object head; |
| 1727 | [super init]; | 1727 | [super init]; |
| 1728 | 1728 | ||
| 1729 | if (XTYPE (contents) == Lisp_Cons) | 1729 | if (CONSP (contents)) |
| 1730 | { | 1730 | { |
| 1731 | head = Fcar (contents); | 1731 | head = Fcar (contents); |
| 1732 | [self process_dialog: Fcdr (contents)]; | 1732 | [self process_dialog: Fcdr (contents)]; |
| @@ -1734,7 +1734,7 @@ ns_popup_dialog (struct frame *f, Lisp_Object header, Lisp_Object contents) | |||
| 1734 | else | 1734 | else |
| 1735 | head = contents; | 1735 | head = contents; |
| 1736 | 1736 | ||
| 1737 | if (XTYPE (head) == Lisp_String) | 1737 | if (STRINGP (head)) |
| 1738 | [title setStringValue: | 1738 | [title setStringValue: |
| 1739 | [NSString stringWithUTF8String: SSDATA (head)]]; | 1739 | [NSString stringWithUTF8String: SSDATA (head)]]; |
| 1740 | else if (isQ == YES) | 1740 | else if (isQ == YES) |