Mercurial > hg > xemacs-beta
comparison src/EmacsFrame.c @ 5077:d372b17f63ce
clean up toolbar/gutter edge geometry
-------------------- ChangeLog entries follow: --------------------
src/ChangeLog addition:
2010-02-25 Ben Wing <ben@xemacs.org>
* EmacsFrame.c (EmacsFrameSetValues):
* frame-impl.h:
* frame-impl.h (struct frame):
* frame-impl.h (FRAME_THEORETICAL_TOP_TOOLBAR_HEIGHT):
* frame-impl.h (FRAME_THEORETICAL_TOP_TOOLBAR_BORDER_WIDTH):
* frame-impl.h (FRAME_REAL_TOP_TOOLBAR_HEIGHT):
* frame-impl.h (FRAME_REAL_TOP_TOOLBAR_BORDER_WIDTH):
* frame-impl.h (FRAME_REAL_TOP_TOOLBAR_VISIBLE):
* frame-impl.h (FRAME_REAL_TOP_TOOLBAR_BOUNDS):
* frame.h:
* frame.h (enum edge_pos):
* gutter.c:
* gutter.c (get_gutter_coords):
* gutter.c (display_boxes_in_gutter_p):
* gutter.c (construct_window_gutter_spec):
* gutter.c (calculate_gutter_size_from_display_lines):
* gutter.c (calculate_gutter_size):
* gutter.c (output_gutter):
* gutter.c (clear_gutter):
* gutter.c (mark_gutters):
* gutter.c (gutter_extent_signal_changed_region_maybe):
* gutter.c (update_gutter_geometry):
* gutter.c (update_frame_gutter_geometry):
* gutter.c (update_frame_gutters):
* gutter.c (reset_gutter_display_lines):
* gutter.c (redraw_exposed_gutter):
* gutter.c (redraw_exposed_gutters):
* gutter.c (free_frame_gutters):
* gutter.c (decode_gutter_position):
* gutter.c (Fset_default_gutter_position):
* gutter.c (Fgutter_pixel_width):
* gutter.c (Fgutter_pixel_height):
* gutter.c (recompute_overlaying_specifier):
* gutter.c (gutter_specs_changed_1):
* gutter.c (gutter_specs_changed):
* gutter.c (top_gutter_specs_changed):
* gutter.c (bottom_gutter_specs_changed):
* gutter.c (left_gutter_specs_changed):
* gutter.c (right_gutter_specs_changed):
* gutter.c (gutter_geometry_changed_in_window):
* gutter.c (init_frame_gutters):
* gutter.c (specifier_vars_of_gutter):
* gutter.h:
* gutter.h (WINDOW_REAL_TOP_GUTTER_BOUNDS):
* gutter.h (FRAME_TOP_GUTTER_BOUNDS):
* lisp.h (enum edge_style):
* native-gtk-toolbar.c:
* native-gtk-toolbar.c (gtk_output_toolbar):
* native-gtk-toolbar.c (gtk_clear_toolbar):
* native-gtk-toolbar.c (gtk_output_frame_toolbars):
* native-gtk-toolbar.c (gtk_initialize_frame_toolbars):
* toolbar-msw.c:
* toolbar-msw.c (TOOLBAR_HANDLE):
* toolbar-msw.c (allocate_toolbar_item_id):
* toolbar-msw.c (mswindows_clear_toolbar):
* toolbar-msw.c (mswindows_output_toolbar):
* toolbar-msw.c (mswindows_move_toolbar):
* toolbar-msw.c (mswindows_redraw_exposed_toolbars):
* toolbar-msw.c (mswindows_initialize_frame_toolbars):
* toolbar-msw.c (mswindows_output_frame_toolbars):
* toolbar-msw.c (mswindows_clear_frame_toolbars):
* toolbar-msw.c (DELETE_TOOLBAR):
* toolbar-msw.c (mswindows_free_frame_toolbars):
* toolbar-msw.c (mswindows_get_toolbar_button_text):
* toolbar-xlike.c:
* toolbar-xlike.c (__prepare_button_area):
* toolbar-xlike.c (XLIKE_OUTPUT_BUTTONS_LOOP):
* toolbar-xlike.c (xlike_output_toolbar):
* toolbar-xlike.c (xlike_clear_toolbar):
* toolbar-xlike.c (xlike_output_frame_toolbars):
* toolbar-xlike.c (xlike_clear_frame_toolbars):
* toolbar-xlike.c (xlike_redraw_exposed_toolbar):
* toolbar-xlike.c (xlike_redraw_exposed_toolbars):
* toolbar-xlike.c (xlike_redraw_frame_toolbars):
* toolbar.c:
* toolbar.c (decode_toolbar_position):
* toolbar.c (Fset_default_toolbar_position):
* toolbar.c (mark_frame_toolbar_buttons_dirty):
* toolbar.c (compute_frame_toolbar_buttons):
* toolbar.c (set_frame_toolbar):
* toolbar.c (compute_frame_toolbars_data):
* toolbar.c (update_frame_toolbars_geometry):
* toolbar.c (init_frame_toolbars):
* toolbar.c (get_toolbar_coords):
* toolbar.c (CHECK_TOOLBAR):
* toolbar.c (toolbar_buttons_at_pixpos):
* toolbar.c (CTB_ERROR):
* toolbar.c (recompute_overlaying_specifier):
* toolbar.c (specifier_vars_of_toolbar):
* toolbar.h:
* toolbar.h (SET_TOOLBAR_WAS_VISIBLE_FLAG):
Create new enum edge_pos with TOP_EDGE, BOTTOM_EDGE, LEFT_EDGE,
RIGHT_EDGE; subsume TOP_BORDER, TOP_GUTTER, enum toolbar_pos,
enum gutter_pos, etc.
Create EDGE_POS_LOOP, subsuming GUTTER_POS_LOOP.
Create NUM_EDGES, use in many places instead of hardcoded '4'.
Instead of top_toolbar_was_visible, bottom_toolbar_was_visible,
etc. make an array toolbar_was_visible[NUM_EDGES]. This increases
the frame size by 15 bytes or so (could be 3 if we use Boolbytes)
but hardly seems w to matter -- frames are heavy weight objects
anyway. Same with top_gutter_was_visible, etc.
Remove duplicated SET_TOOLBAR_WAS_VISIBLE_FLAG and put defn in
one place (toolbar.h).
author | Ben Wing <ben@xemacs.org> |
---|---|
date | Thu, 25 Feb 2010 04:45:13 -0600 |
parents | 92dc90c0bb40 |
children | 97eb4942aec8 |
comparison
equal
deleted
inserted
replaced
5076:d555581e3cba | 5077:d372b17f63ce |
---|---|
409 #endif /* HAVE_SCROLLBARS */ | 409 #endif /* HAVE_SCROLLBARS */ |
410 #ifdef HAVE_TOOLBARS | 410 #ifdef HAVE_TOOLBARS |
411 if (cur->emacs_frame.top_toolbar_height != | 411 if (cur->emacs_frame.top_toolbar_height != |
412 new_->emacs_frame.top_toolbar_height) | 412 new_->emacs_frame.top_toolbar_height) |
413 Fadd_spec_to_specifier | 413 Fadd_spec_to_specifier |
414 (Vtoolbar_size[TOP_TOOLBAR], | 414 (Vtoolbar_size[TOP_EDGE], |
415 make_int (new_->emacs_frame.top_toolbar_height), | 415 make_int (new_->emacs_frame.top_toolbar_height), |
416 wrap_frame (f), Qnil, Qnil); | 416 wrap_frame (f), Qnil, Qnil); |
417 if (cur->emacs_frame.bottom_toolbar_height != | 417 if (cur->emacs_frame.bottom_toolbar_height != |
418 new_->emacs_frame.bottom_toolbar_height) | 418 new_->emacs_frame.bottom_toolbar_height) |
419 Fadd_spec_to_specifier | 419 Fadd_spec_to_specifier |
420 (Vtoolbar_size[BOTTOM_TOOLBAR], | 420 (Vtoolbar_size[BOTTOM_EDGE], |
421 make_int (new_->emacs_frame.bottom_toolbar_height), | 421 make_int (new_->emacs_frame.bottom_toolbar_height), |
422 wrap_frame (f), Qnil, Qnil); | 422 wrap_frame (f), Qnil, Qnil); |
423 if (cur->emacs_frame.left_toolbar_width != | 423 if (cur->emacs_frame.left_toolbar_width != |
424 new_->emacs_frame.left_toolbar_width) | 424 new_->emacs_frame.left_toolbar_width) |
425 Fadd_spec_to_specifier | 425 Fadd_spec_to_specifier |
426 (Vtoolbar_size[LEFT_TOOLBAR], | 426 (Vtoolbar_size[LEFT_EDGE], |
427 make_int (new_->emacs_frame.left_toolbar_width), | 427 make_int (new_->emacs_frame.left_toolbar_width), |
428 wrap_frame (f), Qnil, Qnil); | 428 wrap_frame (f), Qnil, Qnil); |
429 if (cur->emacs_frame.right_toolbar_width != | 429 if (cur->emacs_frame.right_toolbar_width != |
430 new_->emacs_frame.right_toolbar_width) | 430 new_->emacs_frame.right_toolbar_width) |
431 Fadd_spec_to_specifier | 431 Fadd_spec_to_specifier |
432 (Vtoolbar_size[RIGHT_TOOLBAR], | 432 (Vtoolbar_size[RIGHT_EDGE], |
433 make_int (new_->emacs_frame.right_toolbar_width), | 433 make_int (new_->emacs_frame.right_toolbar_width), |
434 wrap_frame (f), Qnil, Qnil); | 434 wrap_frame (f), Qnil, Qnil); |
435 if (cur->emacs_frame.top_toolbar_border_width != | 435 if (cur->emacs_frame.top_toolbar_border_width != |
436 new_->emacs_frame.top_toolbar_border_width) | 436 new_->emacs_frame.top_toolbar_border_width) |
437 Fadd_spec_to_specifier | 437 Fadd_spec_to_specifier |
438 (Vtoolbar_border_width[TOP_TOOLBAR], | 438 (Vtoolbar_border_width[TOP_EDGE], |
439 make_int (new_->emacs_frame.top_toolbar_border_width), | 439 make_int (new_->emacs_frame.top_toolbar_border_width), |
440 wrap_frame (f), Qnil, Qnil); | 440 wrap_frame (f), Qnil, Qnil); |
441 if (cur->emacs_frame.bottom_toolbar_border_width != | 441 if (cur->emacs_frame.bottom_toolbar_border_width != |
442 new_->emacs_frame.bottom_toolbar_border_width) | 442 new_->emacs_frame.bottom_toolbar_border_width) |
443 Fadd_spec_to_specifier | 443 Fadd_spec_to_specifier |
444 (Vtoolbar_border_width[BOTTOM_TOOLBAR], | 444 (Vtoolbar_border_width[BOTTOM_EDGE], |
445 make_int (new_->emacs_frame.bottom_toolbar_border_width), | 445 make_int (new_->emacs_frame.bottom_toolbar_border_width), |
446 wrap_frame (f), Qnil, Qnil); | 446 wrap_frame (f), Qnil, Qnil); |
447 if (cur->emacs_frame.left_toolbar_border_width != | 447 if (cur->emacs_frame.left_toolbar_border_width != |
448 new_->emacs_frame.left_toolbar_border_width) | 448 new_->emacs_frame.left_toolbar_border_width) |
449 Fadd_spec_to_specifier | 449 Fadd_spec_to_specifier |
450 (Vtoolbar_border_width[LEFT_TOOLBAR], | 450 (Vtoolbar_border_width[LEFT_EDGE], |
451 make_int (new_->emacs_frame.left_toolbar_border_width), | 451 make_int (new_->emacs_frame.left_toolbar_border_width), |
452 wrap_frame (f), Qnil, Qnil); | 452 wrap_frame (f), Qnil, Qnil); |
453 if (cur->emacs_frame.right_toolbar_border_width != | 453 if (cur->emacs_frame.right_toolbar_border_width != |
454 new_->emacs_frame.right_toolbar_border_width) | 454 new_->emacs_frame.right_toolbar_border_width) |
455 Fadd_spec_to_specifier | 455 Fadd_spec_to_specifier |
456 (Vtoolbar_border_width[RIGHT_TOOLBAR], | 456 (Vtoolbar_border_width[RIGHT_EDGE], |
457 make_int (new_->emacs_frame.right_toolbar_border_width), | 457 make_int (new_->emacs_frame.right_toolbar_border_width), |
458 wrap_frame (f), Qnil, Qnil); | 458 wrap_frame (f), Qnil, Qnil); |
459 #endif /* HAVE_TOOLBARS */ | 459 #endif /* HAVE_TOOLBARS */ |
460 | 460 |
461 in_resource_setting--; | 461 in_resource_setting--; |