diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/ChangeLog | 16 | ||||
| -rw-r--r-- | src/config.in | 2 | ||||
| -rw-r--r-- | src/dispextern.h | 2 | ||||
| -rw-r--r-- | src/frame.c | 2 | ||||
| -rw-r--r-- | src/image.c | 2 | ||||
| -rw-r--r-- | src/nsfns.m | 16 | ||||
| -rw-r--r-- | src/nsterm.h | 4 | ||||
| -rw-r--r-- | src/s/darwin.h | 4 | ||||
| -rw-r--r-- | src/xdisp.c | 2 |
9 files changed, 33 insertions, 17 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index e360dc48ba8..2c4152d1dd6 100644 --- a/src/ChangeLog +++ b/src/ChangeLog | |||
| @@ -1,3 +1,19 @@ | |||
| 1 | 2008-07-16 Adrian Robert <Adrian.B.Robert@gmail.com> | ||
| 2 | |||
| 3 | * config.in | ||
| 4 | * dispextern.h | ||
| 5 | * frame.c | ||
| 6 | * image.c | ||
| 7 | * nsfns.m | ||
| 8 | * nsfont.m | ||
| 9 | * nsgui.h | ||
| 10 | * nsmenu.m | ||
| 11 | * nsterm.h | ||
| 12 | * nsterm.m | ||
| 13 | * s/darwin.h | ||
| 14 | * xdisp.c: Change most cases of PENDING to TODO, some to FIXME or | ||
| 15 | XXX. | ||
| 16 | |||
| 1 | 2008-07-17 Andreas Schwab <schwab@suse.de> | 17 | 2008-07-17 Andreas Schwab <schwab@suse.de> |
| 2 | 18 | ||
| 3 | * s/cygwin.h (LIB_STANDARD_LIBSRC): Don't define. | 19 | * s/cygwin.h (LIB_STANDARD_LIBSRC): Don't define. |
diff --git a/src/config.in b/src/config.in index d520d50b9b7..5a34ca7046a 100644 --- a/src/config.in +++ b/src/config.in | |||
| @@ -997,7 +997,7 @@ along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>. */ | |||
| 997 | #define CANNOT_DUMP | 997 | #define CANNOT_DUMP |
| 998 | #endif | 998 | #endif |
| 999 | 999 | ||
| 1000 | /* PENDING: These are used for the Carbon port only. */ | 1000 | /* TODO: These are used for the Carbon port only. */ |
| 1001 | #undef MAC_OS | 1001 | #undef MAC_OS |
| 1002 | #undef MAC_OSX | 1002 | #undef MAC_OSX |
| 1003 | 1003 | ||
diff --git a/src/dispextern.h b/src/dispextern.h index 180820d4b80..5c06437f634 100644 --- a/src/dispextern.h +++ b/src/dispextern.h | |||
| @@ -2946,7 +2946,7 @@ struct atimer; /* Defined in atimer.h. */ | |||
| 2946 | an hourglass cursor on all frames. */ | 2946 | an hourglass cursor on all frames. */ |
| 2947 | extern struct atimer *hourglass_atimer; | 2947 | extern struct atimer *hourglass_atimer; |
| 2948 | 2948 | ||
| 2949 | /* Each GUI implements these. PENDING: move into RIF. */ | 2949 | /* Each GUI implements these. FIXME: move into RIF. */ |
| 2950 | extern void show_hourglass P_ ((struct atimer *)); | 2950 | extern void show_hourglass P_ ((struct atimer *)); |
| 2951 | extern void hide_hourglass P_ ((void)); | 2951 | extern void hide_hourglass P_ ((void)); |
| 2952 | 2952 | ||
diff --git a/src/frame.c b/src/frame.c index 88cabc2ac80..21b16154662 100644 --- a/src/frame.c +++ b/src/frame.c | |||
| @@ -2986,7 +2986,7 @@ x_set_frame_parameters (f, alist) | |||
| 2986 | 2986 | ||
| 2987 | old_value = get_frame_param (f, prop); | 2987 | old_value = get_frame_param (f, prop); |
| 2988 | fullscreen_is_being_set |= EQ (prop, Qfullscreen); | 2988 | fullscreen_is_being_set |= EQ (prop, Qfullscreen); |
| 2989 | #ifndef HAVE_NS /* PENDING: ensure font attrs change goes through */ | 2989 | #ifndef HAVE_NS /* XXX: ensure font attrs change goes through */ |
| 2990 | if (NILP (Fequal (val, old_value))) | 2990 | if (NILP (Fequal (val, old_value))) |
| 2991 | #endif | 2991 | #endif |
| 2992 | { | 2992 | { |
diff --git a/src/image.c b/src/image.c index 1d73704b388..2685237e718 100644 --- a/src/image.c +++ b/src/image.c | |||
| @@ -435,7 +435,7 @@ XGetImage (Display *display, Pixmap pixmap, int x, int y, | |||
| 435 | unsigned int width, unsigned int height, | 435 | unsigned int width, unsigned int height, |
| 436 | unsigned long plane_mask, int format) | 436 | unsigned long plane_mask, int format) |
| 437 | { | 437 | { |
| 438 | /* PENDING: not sure what this function is supposed to do.. */ | 438 | /* TODO: not sure what this function is supposed to do.. */ |
| 439 | ns_retain_object(pixmap); | 439 | ns_retain_object(pixmap); |
| 440 | return pixmap; | 440 | return pixmap; |
| 441 | } | 441 | } |
diff --git a/src/nsfns.m b/src/nsfns.m index 8ec6518aead..1752512d2dc 100644 --- a/src/nsfns.m +++ b/src/nsfns.m | |||
| @@ -742,7 +742,7 @@ ns_set_menu_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval) | |||
| 742 | } | 742 | } |
| 743 | 743 | ||
| 744 | 744 | ||
| 745 | /* 23: PENDING: there is an erroneous direct call in window.c to this fn */ | 745 | /* 23: XXX: there is an erroneous direct call in window.c to this fn */ |
| 746 | void | 746 | void |
| 747 | x_set_menu_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval) | 747 | x_set_menu_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval) |
| 748 | { | 748 | { |
| @@ -783,7 +783,7 @@ ns_set_tool_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval) | |||
| 783 | } | 783 | } |
| 784 | 784 | ||
| 785 | 785 | ||
| 786 | /* 23: PENDING: there is an erroneous direct call in window.c to this fn */ | 786 | /* 23: XXX: there is an erroneous direct call in window.c to this fn */ |
| 787 | void | 787 | void |
| 788 | x_set_tool_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval) | 788 | x_set_tool_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval) |
| 789 | { | 789 | { |
| @@ -912,7 +912,7 @@ XParseGeometry (char *string, int *x, int *y, | |||
| 912 | } | 912 | } |
| 913 | 913 | ||
| 914 | 914 | ||
| 915 | /*PENDING: move to nsterm? */ | 915 | /* TODO: move to nsterm? */ |
| 916 | int | 916 | int |
| 917 | ns_lisp_to_cursor_type (Lisp_Object arg) | 917 | ns_lisp_to_cursor_type (Lisp_Object arg) |
| 918 | { | 918 | { |
| @@ -1213,7 +1213,7 @@ be shared by the new frame.") | |||
| 1213 | "background", "Background", RES_TYPE_STRING); | 1213 | "background", "Background", RES_TYPE_STRING); |
| 1214 | x_default_parameter (f, parms, Qcursor_color, build_string ("grey"), | 1214 | x_default_parameter (f, parms, Qcursor_color, build_string ("grey"), |
| 1215 | "cursorColor", "CursorColor", RES_TYPE_STRING); | 1215 | "cursorColor", "CursorColor", RES_TYPE_STRING); |
| 1216 | /*PENDING: not suppported yet in NS */ | 1216 | /* FIXME: not suppported yet in NS */ |
| 1217 | x_default_parameter (f, parms, Qline_spacing, Qnil, | 1217 | x_default_parameter (f, parms, Qline_spacing, Qnil, |
| 1218 | "lineSpacing", "LineSpacing", RES_TYPE_NUMBER); | 1218 | "lineSpacing", "LineSpacing", RES_TYPE_NUMBER); |
| 1219 | x_default_parameter (f, parms, Qleft_fringe, Qnil, | 1219 | x_default_parameter (f, parms, Qleft_fringe, Qnil, |
| @@ -1233,7 +1233,7 @@ be shared by the new frame.") | |||
| 1233 | x_default_parameter (f, parms, Qtitle, Qnil, "title", "Title", | 1233 | x_default_parameter (f, parms, Qtitle, Qnil, "title", "Title", |
| 1234 | RES_TYPE_STRING); | 1234 | RES_TYPE_STRING); |
| 1235 | 1235 | ||
| 1236 | /*PENDING: other terms seem to get away w/o this complexity.. */ | 1236 | /* TODO: other terms seem to get away w/o this complexity.. */ |
| 1237 | if (NILP (Fassq (Qwidth, parms))) | 1237 | if (NILP (Fassq (Qwidth, parms))) |
| 1238 | { | 1238 | { |
| 1239 | Lisp_Object value | 1239 | Lisp_Object value |
| @@ -1610,7 +1610,7 @@ If omitted or nil, that stands for the selected frame's display.") | |||
| 1610 | (display) | 1610 | (display) |
| 1611 | Lisp_Object display; | 1611 | Lisp_Object display; |
| 1612 | { | 1612 | { |
| 1613 | /*PENDING: return GUI version on GNUSTEP, ?? on OS X */ | 1613 | /* FIXME: return GUI version on GNUSTEP, ?? on OS X */ |
| 1614 | return build_string ("1.0"); | 1614 | return build_string ("1.0"); |
| 1615 | } | 1615 | } |
| 1616 | 1616 | ||
| @@ -2108,7 +2108,7 @@ x_set_scroll_bar_default_width (f) | |||
| 2108 | const char * | 2108 | const char * |
| 2109 | x_get_string_resource (XrmDatabase rdb, char *name, char *class) | 2109 | x_get_string_resource (XrmDatabase rdb, char *name, char *class) |
| 2110 | { | 2110 | { |
| 2111 | /* remove appname prefix; PENDING: allow for !="Emacs" */ | 2111 | /* remove appname prefix; TODO: allow for !="Emacs" */ |
| 2112 | char *toCheck = class + (!strncmp (class, "Emacs.", 6) ? 6 : 0); | 2112 | char *toCheck = class + (!strncmp (class, "Emacs.", 6) ? 6 : 0); |
| 2113 | const char *res; | 2113 | const char *res; |
| 2114 | check_ns (); | 2114 | check_ns (); |
| @@ -2359,7 +2359,7 @@ If omitted or nil, that stands for the selected frame's display.") | |||
| 2359 | /* Unused dummy def needed for compatibility. */ | 2359 | /* Unused dummy def needed for compatibility. */ |
| 2360 | Lisp_Object tip_frame; | 2360 | Lisp_Object tip_frame; |
| 2361 | 2361 | ||
| 2362 | /*PENDING: move to xdisp or similar */ | 2362 | /* TODO: move to xdisp or similar */ |
| 2363 | static void | 2363 | static void |
| 2364 | compute_tip_xy (f, parms, dx, dy, width, height, root_x, root_y) | 2364 | compute_tip_xy (f, parms, dx, dy, width, height, root_x, root_y) |
| 2365 | struct frame *f; | 2365 | struct frame *f; |
diff --git a/src/nsterm.h b/src/nsterm.h index 260263549b6..2a70ae31d53 100644 --- a/src/nsterm.h +++ b/src/nsterm.h | |||
| @@ -542,7 +542,7 @@ extern struct ns_display_info *ns_display_list; | |||
| 542 | extern Lisp_Object ns_display_name_list; | 542 | extern Lisp_Object ns_display_name_list; |
| 543 | extern struct ns_display_info *ns_display_info_for_name (); | 543 | extern struct ns_display_info *ns_display_info_for_name (); |
| 544 | 544 | ||
| 545 | /* 23: PENDING: these functions (we defined in nsfns) are used in various | 545 | /* 23: FIXME: these functions (we defined in nsfns) are used in various |
| 546 | places, but no prototypes are provided */ | 546 | places, but no prototypes are provided */ |
| 547 | struct ns_display_info *check_x_display_info (Lisp_Object frame); | 547 | struct ns_display_info *check_x_display_info (Lisp_Object frame); |
| 548 | FRAME_PTR check_x_frame (Lisp_Object frame); | 548 | FRAME_PTR check_x_frame (Lisp_Object frame); |
| @@ -679,7 +679,7 @@ struct x_output | |||
| 679 | (FRAME_SCROLL_BAR_COLS (f) * FRAME_COLUMN_WIDTH (f) \ | 679 | (FRAME_SCROLL_BAR_COLS (f) * FRAME_COLUMN_WIDTH (f) \ |
| 680 | - NS_SCROLL_BAR_WIDTH (f)) : 0) | 680 | - NS_SCROLL_BAR_WIDTH (f)) : 0) |
| 681 | 681 | ||
| 682 | /*PENDING: fix for GNUstep inconsistent accounting for titlebar */ | 682 | /* XXX: fix for GNUstep inconsistent accounting for titlebar */ |
| 683 | #ifdef NS_IMPL_GNUSTEP | 683 | #ifdef NS_IMPL_GNUSTEP |
| 684 | #define NS_TOP_POS(f) ((f)->top_pos + 18) | 684 | #define NS_TOP_POS(f) ((f)->top_pos + 18) |
| 685 | #else | 685 | #else |
diff --git a/src/s/darwin.h b/src/s/darwin.h index 4dbaa8ff3c6..9fcf2ca6dff 100644 --- a/src/s/darwin.h +++ b/src/s/darwin.h | |||
| @@ -206,8 +206,8 @@ along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>. */ | |||
| 206 | 206 | ||
| 207 | /* Link in the Carbon or AppKit lib. */ | 207 | /* Link in the Carbon or AppKit lib. */ |
| 208 | #ifdef HAVE_NS | 208 | #ifdef HAVE_NS |
| 209 | /* PENDING: lresolv is here because configure when testing #undefs res_init, | 209 | /* XXX: lresolv is here because configure when testing #undefs res_init, |
| 210 | a macro in /usr/include/resolv.h for res_9_init, not in stdc lib. */ | 210 | a macro in /usr/include/resolv.h for res_9_init, not in stdc lib. */ |
| 211 | #define LIBS_MACGUI -framework AppKit -lresolv | 211 | #define LIBS_MACGUI -framework AppKit -lresolv |
| 212 | #define SYSTEM_PURESIZE_EXTRA 200000 | 212 | #define SYSTEM_PURESIZE_EXTRA 200000 |
| 213 | #define HEADERPAD_EXTRA 6C8 | 213 | #define HEADERPAD_EXTRA 6C8 |
diff --git a/src/xdisp.c b/src/xdisp.c index d7b79b527cd..84908f71cf4 100644 --- a/src/xdisp.c +++ b/src/xdisp.c | |||
| @@ -11547,7 +11547,7 @@ redisplay_internal (preserve_echo_area) | |||
| 11547 | /* Resized active mini-window to fit the size of what it is | 11547 | /* Resized active mini-window to fit the size of what it is |
| 11548 | showing if its contents might have changed. */ | 11548 | showing if its contents might have changed. */ |
| 11549 | must_finish = 1; | 11549 | must_finish = 1; |
| 11550 | /* PENDING: this causes all frames to be updated, which seems unnecessary | 11550 | /* FIXME: this causes all frames to be updated, which seems unnecessary |
| 11551 | since only the current frame needs to be considered. This function needs | 11551 | since only the current frame needs to be considered. This function needs |
| 11552 | to be rewritten with two variables, consider_all_windows and | 11552 | to be rewritten with two variables, consider_all_windows and |
| 11553 | consider_all_frames. */ | 11553 | consider_all_frames. */ |