Mercurial > emacs
annotate admin/notes/iftc @ 105923:1011707400d3
* menu.c (Fx_popup_menu): Consolidate versions from xmenu.c,
w32menu.c, and nsmenu.m.
Simplify the obsolete case where position is nil.
(cleanup_popup_menu): New function, moved from nsmenu.m.
(struct skp): Remove slot `notreal'.
(single_keymap_panes, keymap_panes): Remove arg `notreal' and adjust callers.
(single_menu_item): Adjust call to parse_menu_item.
(syms_of_menu): Defsubr x-popup-menu.
* menu.h (Vmenu_updating_frame): Consolidate declarations from *menu.c.
(keymap_panes): Don't export any more.
(mouse_position_for_popup, w32_menu_show, ns_menu_show, xmenu_show): Declare.
* keyboard.c (parse_menu_item): Remove arg `notreal'.
(menu_bar_item, read_char_minibuf_menu_prompt): Adjust callers.
* keyboard.h (parse_menu_item): Update declaration.
* xmenu.c (Fx_popup_menu): Remove.
(syms_of_xmenu): Don't defsubr x-popup-menu.
* w32menu.c (Fx_popup_menu): Remove.
(syms_of_w32menu): Don't defsubr x-popup-menu.
* nsmenu.m (cleanup_popup_menu): Remove.
(ns_menu_show): Rename from ns_popup_menu and remove all the code
moved to menu.c's Fx_popup_menu.
(Fx_popup_menu): Remove.
(syms_of_nsmenu): Don't defsubr x-popup-menu, and don't initialize
menu_items (it's done in menu.c already).
author | Stefan Monnier <monnier@iro.umontreal.ca> |
---|---|
date | Mon, 09 Nov 2009 06:21:03 +0000 |
parents | 695cf19ef79e |
children | 375f2633d815 ef719132ddfa |
rev | line source |
---|---|
45625 | 1 Iso-Functional Type Contour |
2 | |
3 | |
4 This is a term coined to describe "column int->float" change approach, and can | |
5 be used whenever low-level types need to change (hopefully not often!) but the | |
6 meanings of the values (whose type has changed) do not. | |
7 | |
8 The premise is that changing a low-level type potentially means lots of code | |
9 needs to be changed as well, and the question is how to do this incrementally, | |
10 which is the preferred way to change things. | |
11 | |
12 Say LOW and HIGH are C functions: | |
13 | |
14 int LOW (void) { return 1; } | |
15 void HIGH (void) { int value = LOW (); } | |
16 | |
17 We want to convert LOW to return float, so we cast HIGH usage: | |
18 | |
19 float LOW (void) { return 1.0; } | |
20 void HIGH (void) { int value = (int) LOW (); } /* iftc */ | |
21 | |
22 The comment /* iftc */ is used to mark this type of casting to differentiate | |
23 it from other casting. We commit the changes and can now go about modifying | |
24 LOW and HIGH separately. When HIGH is ready to handle the type change, the | |
25 cast can be removed. | |
52401 | 26 |
27 ;;; arch-tag: 3309cc41-5d59-421b-b7be-c94b04083bb5 |