diff options
| author | Paul Eggert | 2018-07-19 22:03:34 -0700 |
|---|---|---|
| committer | Paul Eggert | 2018-07-19 22:05:27 -0700 |
| commit | fb24ce37d1bc258cfc3884d9828aa0602fa06e1d (patch) | |
| tree | 2c0669e13f1324d78b9a0e2de496fc366016ef12 /src/frame.c | |
| parent | 96d77f9eb882b68e994e187ed9c2156a23e3279d (diff) | |
| download | emacs-fb24ce37d1bc258cfc3884d9828aa0602fa06e1d.tar.gz emacs-fb24ce37d1bc258cfc3884d9828aa0602fa06e1d.zip | |
Prefer NILP (x) to EQ (x, Qnil)
This simplifies the code a bit, and also simplifies some
potential future changes slightly (e.g., altering eq vs eql).
* src/alloc.c (mark_object):
* src/callint.c (fix_command):
* src/chartab.c (Fchar_table_range, Fset_char_table_range):
* src/dbusbind.c (XD_OBJECT_TO_DBUS_TYPE, xd_signature):
* src/dired.c (Fsystem_users):
* src/fileio.c (Fdo_auto_save):
* src/fns.c (concat):
* src/frame.c (get_frame_param, frame_inhibit_resize)
(store_in_alist, store_frame_param, x_set_autoraise)
(x_set_autolower, x_get_arg):
* src/image.c (Fclear_image_cache):
* src/intervals.c (intervals_equal):
* src/intervals.h (DEFAULT_INTERVAL_P):
* src/lread.c (substitute_object_recurse):
* src/menu.c (digest_single_submenu)
(find_and_call_menu_selection)
(find_and_return_menu_selection):
* src/nsfns.m (x_set_icon_name, Fx_create_frame):
* src/nsmenu.m (ns_menu_show):
* src/nsselect.m (ns_string_to_pasteboard_internal)
(Fns_selection_exists_p, Fns_selection_owner_p):
* src/process.c (Faccept_process_output)
(wait_reading_process_output):
* src/terminal.c (store_terminal_param):
* src/textprop.c (verify_interval_modification):
* src/xdisp.c (next_element_from_buffer):
* src/xfaces.c (Finternal_set_lisp_face_attribute):
* src/xfns.c (x_set_icon_type, Fx_synchronize):
* src/xmenu.c (x_menu_show):
* src/xselect.c (Fx_selection_owner_p)
(Fx_selection_exists_p):
* src/xwidget.c (xwidget_view_lookup):
Prefer NILP (x) to EQ (x, Qnil).
Diffstat (limited to 'src/frame.c')
| -rw-r--r-- | src/frame.c | 27 |
1 files changed, 10 insertions, 17 deletions
diff --git a/src/frame.c b/src/frame.c index d477c1acc3f..85ec7401d6e 100644 --- a/src/frame.c +++ b/src/frame.c | |||
| @@ -139,14 +139,9 @@ check_window_system (struct frame *f) | |||
| 139 | /* Return the value of frame parameter PROP in frame FRAME. */ | 139 | /* Return the value of frame parameter PROP in frame FRAME. */ |
| 140 | 140 | ||
| 141 | Lisp_Object | 141 | Lisp_Object |
| 142 | get_frame_param (register struct frame *frame, Lisp_Object prop) | 142 | get_frame_param (struct frame *frame, Lisp_Object prop) |
| 143 | { | 143 | { |
| 144 | register Lisp_Object tem; | 144 | return Fcdr (Fassq (prop, frame->param_alist)); |
| 145 | |||
| 146 | tem = Fassq (prop, frame->param_alist); | ||
| 147 | if (EQ (tem, Qnil)) | ||
| 148 | return tem; | ||
| 149 | return Fcdr (tem); | ||
| 150 | } | 145 | } |
| 151 | 146 | ||
| 152 | 147 | ||
| @@ -189,9 +184,9 @@ frame_inhibit_resize (struct frame *f, bool horizontal, Lisp_Object parameter) | |||
| 189 | || (CONSP (frame_inhibit_implied_resize) | 184 | || (CONSP (frame_inhibit_implied_resize) |
| 190 | && !NILP (Fmemq (parameter, frame_inhibit_implied_resize))) | 185 | && !NILP (Fmemq (parameter, frame_inhibit_implied_resize))) |
| 191 | || (horizontal | 186 | || (horizontal |
| 192 | && !EQ (fullscreen, Qnil) && !EQ (fullscreen, Qfullheight)) | 187 | && !NILP (fullscreen) && !EQ (fullscreen, Qfullheight)) |
| 193 | || (!horizontal | 188 | || (!horizontal |
| 194 | && !EQ (fullscreen, Qnil) && !EQ (fullscreen, Qfullwidth)) | 189 | && !NILP (fullscreen) && !EQ (fullscreen, Qfullwidth)) |
| 195 | || FRAME_TERMCAP_P (f) || FRAME_MSDOS_P (f)) | 190 | || FRAME_TERMCAP_P (f) || FRAME_MSDOS_P (f)) |
| 196 | : ((horizontal && f->inhibit_horizontal_resize) | 191 | : ((horizontal && f->inhibit_horizontal_resize) |
| 197 | || (!horizontal && f->inhibit_vertical_resize))); | 192 | || (!horizontal && f->inhibit_vertical_resize))); |
| @@ -2808,10 +2803,8 @@ frames_discard_buffer (Lisp_Object buffer) | |||
| 2808 | void | 2803 | void |
| 2809 | store_in_alist (Lisp_Object *alistptr, Lisp_Object prop, Lisp_Object val) | 2804 | store_in_alist (Lisp_Object *alistptr, Lisp_Object prop, Lisp_Object val) |
| 2810 | { | 2805 | { |
| 2811 | register Lisp_Object tem; | 2806 | Lisp_Object tem = Fassq (prop, *alistptr); |
| 2812 | 2807 | if (NILP (tem)) | |
| 2813 | tem = Fassq (prop, *alistptr); | ||
| 2814 | if (EQ (tem, Qnil)) | ||
| 2815 | *alistptr = Fcons (Fcons (prop, val), *alistptr); | 2808 | *alistptr = Fcons (Fcons (prop, val), *alistptr); |
| 2816 | else | 2809 | else |
| 2817 | Fsetcdr (tem, val); | 2810 | Fsetcdr (tem, val); |
| @@ -2975,7 +2968,7 @@ store_frame_param (struct frame *f, Lisp_Object prop, Lisp_Object val) | |||
| 2975 | 2968 | ||
| 2976 | /* Update the frame parameter alist. */ | 2969 | /* Update the frame parameter alist. */ |
| 2977 | old_alist_elt = Fassq (prop, f->param_alist); | 2970 | old_alist_elt = Fassq (prop, f->param_alist); |
| 2978 | if (EQ (old_alist_elt, Qnil)) | 2971 | if (NILP (old_alist_elt)) |
| 2979 | fset_param_alist (f, Fcons (Fcons (prop, val), f->param_alist)); | 2972 | fset_param_alist (f, Fcons (Fcons (prop, val), f->param_alist)); |
| 2980 | else | 2973 | else |
| 2981 | Fsetcdr (old_alist_elt, val); | 2974 | Fsetcdr (old_alist_elt, val); |
| @@ -4516,13 +4509,13 @@ x_set_visibility (struct frame *f, Lisp_Object value, Lisp_Object oldval) | |||
| 4516 | void | 4509 | void |
| 4517 | x_set_autoraise (struct frame *f, Lisp_Object arg, Lisp_Object oldval) | 4510 | x_set_autoraise (struct frame *f, Lisp_Object arg, Lisp_Object oldval) |
| 4518 | { | 4511 | { |
| 4519 | f->auto_raise = !EQ (Qnil, arg); | 4512 | f->auto_raise = !NILP (arg); |
| 4520 | } | 4513 | } |
| 4521 | 4514 | ||
| 4522 | void | 4515 | void |
| 4523 | x_set_autolower (struct frame *f, Lisp_Object arg, Lisp_Object oldval) | 4516 | x_set_autolower (struct frame *f, Lisp_Object arg, Lisp_Object oldval) |
| 4524 | { | 4517 | { |
| 4525 | f->auto_lower = !EQ (Qnil, arg); | 4518 | f->auto_lower = !NILP (arg); |
| 4526 | } | 4519 | } |
| 4527 | 4520 | ||
| 4528 | void | 4521 | void |
| @@ -4973,7 +4966,7 @@ x_get_arg (Display_Info *dpyinfo, Lisp_Object alist, Lisp_Object param, | |||
| 4973 | 4966 | ||
| 4974 | /* If it wasn't specified in ALIST or the Lisp-level defaults, | 4967 | /* If it wasn't specified in ALIST or the Lisp-level defaults, |
| 4975 | look in the X resources. */ | 4968 | look in the X resources. */ |
| 4976 | if (EQ (tem, Qnil)) | 4969 | if (NILP (tem)) |
| 4977 | { | 4970 | { |
| 4978 | if (attribute && dpyinfo) | 4971 | if (attribute && dpyinfo) |
| 4979 | { | 4972 | { |