diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2014-03-04 22:28:20 -0800 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2014-03-04 22:28:20 -0800 |
| commit | d2f720acf5dbf40a838df4b09cd410034d0f6149 (patch) | |
| tree | 2c7a8cd37af9634bc5f49da4444174becb41c19e /src/widget.c | |
| parent | 59587ee4559e791138ea55181834ca4cdd4e8270 (diff) | |
| download | emacs-d2f720acf5dbf40a838df4b09cd410034d0f6149.tar.gz | |
Fix minor --enable-gcc-warnings issues.
* widget.c (update_various_frame_slots, EmacsFrameResize):
Avoid unused locals. Prefer 'if' to '#if' when either will do.
Diffstat (limited to 'src/widget.c')
| -rw-r--r-- | src/widget.c | 25 |
1 files changed, 12 insertions, 13 deletions
diff --git a/src/widget.c b/src/widget.c index 838e775f918..bd0fe826e68 100644 --- a/src/widget.c +++ b/src/widget.c @@ -598,16 +598,18 @@ static void update_various_frame_slots (EmacsFrame ew) { struct frame *f = ew->emacs_frame.frame; - struct x_output *x = f->output_data.x; + /* Don't do that: It confuses the check in change_frame_size_1 whether the pixel size of the frame changed due to a change of the internal border width. Bug#16736. */ -#if 0 - FRAME_PIXEL_HEIGHT (f) = ew->core.height + x->menubar_height; - FRAME_PIXEL_WIDTH (f) = ew->core.width; -#endif - f->internal_border_width = ew->emacs_frame.internal_border_width; + if (false) + { + struct x_output *x = f->output_data.x; + FRAME_PIXEL_HEIGHT (f) = ew->core.height + x->menubar_height; + FRAME_PIXEL_WIDTH (f) = ew->core.width; + } + f->internal_border_width = ew->emacs_frame.internal_border_width; } static void @@ -682,13 +684,11 @@ EmacsFrameResize (Widget widget) { EmacsFrame ew = (EmacsFrame)widget; struct frame *f = ew->emacs_frame.frame; - struct x_output *x = f->output_data.x; -#if 0 /* Always process resize requests pixelwise. Frame maximizing - should work even when frame_resize_pixelwise is nil. */ - if (frame_resize_pixelwise) + /* Always process resize requests pixelwise. Frame maximizing + should work even when frame_resize_pixelwise is nil. */ + if (true || frame_resize_pixelwise) { -#endif /* 0 */ int width, height; pixel_to_text_size (ew, ew->core.width, ew->core.height, &width, &height); @@ -698,10 +698,10 @@ EmacsFrameResize (Widget widget) update_various_frame_slots (ew); cancel_mouse_face (f); -#if 0 /* See comment above. */ } else { + struct x_output *x = f->output_data.x; int columns, rows; pixel_to_char_size (ew, ew->core.width, ew->core.height, &columns, &rows); @@ -717,7 +717,6 @@ EmacsFrameResize (Widget widget) cancel_mouse_face (f); } } -#endif /* 0 */ } static Boolean |