diff options
| author | Chong Yidong | 2008-07-07 20:39:00 +0000 |
|---|---|---|
| committer | Chong Yidong | 2008-07-07 20:39:00 +0000 |
| commit | a6f75881406e540517fd96a8ad28ec997c2127fa (patch) | |
| tree | 7162eacdc76282eeb8bd3b6907168796ab3e9f3d /src | |
| parent | 4d2d5f5fac91254f0d3dc9338c6615ef94290c46 (diff) | |
| download | emacs-a6f75881406e540517fd96a8ad28ec997c2127fa.tar.gz emacs-a6f75881406e540517fd96a8ad28ec997c2127fa.zip | |
(Qinhibit_face_set_after_frame_default): Var deleted.
(x_set_frame_parameters): Don't bind it.
Diffstat (limited to 'src')
| -rw-r--r-- | src/frame.c | 36 |
1 files changed, 6 insertions, 30 deletions
diff --git a/src/frame.c b/src/frame.c index ba7e396ea43..a4fa29bff49 100644 --- a/src/frame.c +++ b/src/frame.c | |||
| @@ -120,7 +120,6 @@ Lisp_Object Qfullscreen, Qfullwidth, Qfullheight, Qfullboth; | |||
| 120 | Lisp_Object Qfont_backend; | 120 | Lisp_Object Qfont_backend; |
| 121 | Lisp_Object Qalpha; | 121 | Lisp_Object Qalpha; |
| 122 | 122 | ||
| 123 | Lisp_Object Qinhibit_face_set_after_frame_default; | ||
| 124 | Lisp_Object Qface_set_after_frame_default; | 123 | Lisp_Object Qface_set_after_frame_default; |
| 125 | 124 | ||
| 126 | Lisp_Object Vterminal_frame; | 125 | Lisp_Object Vterminal_frame; |
| @@ -2967,20 +2966,12 @@ x_set_frame_parameters (f, alist) | |||
| 2967 | || EQ (prop, Qfullscreen)) | 2966 | || EQ (prop, Qfullscreen)) |
| 2968 | { | 2967 | { |
| 2969 | register Lisp_Object param_index, old_value; | 2968 | register Lisp_Object param_index, old_value; |
| 2970 | int count = SPECPDL_INDEX (); | ||
| 2971 | 2969 | ||
| 2972 | old_value = get_frame_param (f, prop); | 2970 | old_value = get_frame_param (f, prop); |
| 2973 | fullscreen_is_being_set |= EQ (prop, Qfullscreen); | 2971 | fullscreen_is_being_set |= EQ (prop, Qfullscreen); |
| 2974 | 2972 | ||
| 2975 | if (NILP (Fequal (val, old_value))) | 2973 | if (NILP (Fequal (val, old_value))) |
| 2976 | { | 2974 | { |
| 2977 | /* For :font attributes, the frame_parm_handler | ||
| 2978 | x_set_font calls `face-set-after-frame-default'. | ||
| 2979 | Unless we bind inhibit-face-set-after-frame-default | ||
| 2980 | here, this would reset the :font attribute that we | ||
| 2981 | just applied to the default value for new faces. */ | ||
| 2982 | specbind (Qinhibit_face_set_after_frame_default, Qt); | ||
| 2983 | |||
| 2984 | store_frame_param (f, prop, val); | 2975 | store_frame_param (f, prop, val); |
| 2985 | 2976 | ||
| 2986 | param_index = Fget (prop, Qx_frame_parameter); | 2977 | param_index = Fget (prop, Qx_frame_parameter); |
| @@ -2989,7 +2980,6 @@ x_set_frame_parameters (f, alist) | |||
| 2989 | < sizeof (frame_parms)/sizeof (frame_parms[0])) | 2980 | < sizeof (frame_parms)/sizeof (frame_parms[0])) |
| 2990 | && FRAME_RIF (f)->frame_parm_handlers[XINT (param_index)]) | 2981 | && FRAME_RIF (f)->frame_parm_handlers[XINT (param_index)]) |
| 2991 | (*(FRAME_RIF (f)->frame_parm_handlers[XINT (param_index)])) (f, val, old_value); | 2982 | (*(FRAME_RIF (f)->frame_parm_handlers[XINT (param_index)])) (f, val, old_value); |
| 2992 | unbind_to (count, Qnil); | ||
| 2993 | } | 2983 | } |
| 2994 | } | 2984 | } |
| 2995 | } | 2985 | } |
| @@ -3392,22 +3382,12 @@ x_set_font (f, arg, oldval) | |||
| 3392 | 3382 | ||
| 3393 | do_pending_window_change (0); | 3383 | do_pending_window_change (0); |
| 3394 | 3384 | ||
| 3395 | /* Don't call `face-set-after-frame-default' when faces haven't been | 3385 | /* We used to call face-set-after-frame-default here, but it leads to |
| 3396 | initialized yet. This is the case when called from | 3386 | recursive calls (since that function can set the `default' face's |
| 3397 | Fx_create_frame. In that case, the X widget or window doesn't | 3387 | font which in turns changes the frame's `font' parameter). |
| 3398 | exist either, and we can end up in x_report_frame_params with a | 3388 | Also I don't know what this call is meant to do, but it seems the |
| 3399 | null widget which gives a segfault. */ | 3389 | wrong way to do it anyway (it does a lot more work than what seems |
| 3400 | if (FRAME_FACE_CACHE (f)) | 3390 | reasonable in response to a change to `font'). */ |
| 3401 | { | ||
| 3402 | XSETFRAME (frame, f); | ||
| 3403 | /* We used to call face-set-after-frame-default here, but it leads to | ||
| 3404 | recursive calls (since that function can set the `default' face's | ||
| 3405 | font which in turns changes the frame's `font' parameter). | ||
| 3406 | Also I don't know what this call is meant to do, but it seems the | ||
| 3407 | wrong way to do it anyway (it does a lot more work than what seems | ||
| 3408 | reasonable in response to a change to `font'). */ | ||
| 3409 | /* call1 (Qface_set_after_frame_default, frame); */ | ||
| 3410 | } | ||
| 3411 | } | 3391 | } |
| 3412 | 3392 | ||
| 3413 | 3393 | ||
| @@ -4410,10 +4390,6 @@ syms_of_frame () | |||
| 4410 | Qface_set_after_frame_default = intern ("face-set-after-frame-default"); | 4390 | Qface_set_after_frame_default = intern ("face-set-after-frame-default"); |
| 4411 | staticpro (&Qface_set_after_frame_default); | 4391 | staticpro (&Qface_set_after_frame_default); |
| 4412 | 4392 | ||
| 4413 | Qinhibit_face_set_after_frame_default | ||
| 4414 | = intern ("inhibit-face-set-after-frame-default"); | ||
| 4415 | staticpro (&Qinhibit_face_set_after_frame_default); | ||
| 4416 | |||
| 4417 | Qfullwidth = intern ("fullwidth"); | 4393 | Qfullwidth = intern ("fullwidth"); |
| 4418 | staticpro (&Qfullwidth); | 4394 | staticpro (&Qfullwidth); |
| 4419 | Qfullheight = intern ("fullheight"); | 4395 | Qfullheight = intern ("fullheight"); |