comparison src/device-gtk.c @ 4477:e34711681f30

Don't determine whether to call general device-type code at startup, rather decide in the device-specific code itself. lisp/ChangeLog addition: 2008-07-07 Aidan Kehoe <kehoea@parhasard.net> Patch to make it up to the device-specific code whether various Lisp functions should be called during device creation, not relying on the startup code to decide this. Also, rename initial-window-system to initial-device-type (which makes more sense in this scheme), always set it. * startup.el (command-line): Use initial-device-type, not initial-window-system; just call #'make-device, leave the special behaviour to be done the first time a console type is initialised to be decided on by the respective console code. * x-init.el (x-app-defaults-directory): Declare that it should be bound. (x-define-dead-key): Have the macro take a DEVICE argument. (x-initialize-compose): Have the function take a DEVICE argument, and use it when checking if various keysyms are available on the keyboard. (x-initialize-keyboard): Have the function take a DEVICE argument, allowing device-specific keyboard initialisation. (make-device-early-x-entry-point-called-p): New. (make-device-late-x-entry-point-called-p): New. Rename pre-x-win-initted, x-win-initted. (make-device-early-x-entry-point): Rename init-pre-x-win, take the call to make-x-device out (it should be called from the device-creation code, not vice-versa). (make-device-late-x-entry-point): Rename init-post-x-win, have it take a DEVICE argument, use that DEVICE argument when working out what device-specific things need doing. Don't use create-console-hook in core code. * x-win-xfree86.el (x-win-init-xfree86): Take a DEVICE argument; use it. * x-win-sun.el (x-win-init-sun): Take a DEVICE argument; use it. * mule/mule-x-init.el: Remove #'init-mule-x-win, an empty function. * tty-init.el (make-device-early-tty-entry-point-called-p): New. Rename pre-tty-win-initted. (make-device-early-tty-entry-point): New. Rename init-pre-tty-win. (make-frame-after-init-entry-point): New. Rename init-post-tty-win to better reflect when it's called. * gtk-init.el (gtk-early-lisp-options-file): New. Move this path to a documented variable. (gtk-command-switch-alist): Wrap the docstring to fewer than 79 columns. (make-device-early-gtk-entry-point-called-p): New. (make-device-late-gtk-entry-point-called-p): New. Renamed gtk-pre-win-initted, gtk-post-win-initted to these. (make-device-early-gtk-entry-point): New. (make-device-late-gtk-entry-point): New. Renamed init-pre-gtk-win, init-post-gtk-win to these. Have make-device-late-gtk-entry-point take a device argument, and use it; have make-device-early-gtk-entry-point load the GTK-specific startup code, instead of doing that in C. (init-gtk-win): Deleted, functionality moved to the GTK device creation code. (gtk-define-dead-key): Have it take a DEVICE argument; use this argument. (gtk-initialize-compose): Ditto. * coding.el (set-terminal-coding-system): Correct the docstring; the function isn't broken. src/ChangeLog addition: 2008-07-07 Aidan Kehoe <kehoea@parhasard.net> Patch to make it up to the device-specific code whether various Lisp functions should be called during device creation, not relying on the startup code to decide this. Also, rename initial-window-system to initial-device-type (which makes more sense in this scheme), always set it. * redisplay.c (Vinitial_device_type): New. (Vinitial_window_system): Removed. Rename initial-window-system to initial-device type, making it a stream if we're noninteractive. Update its docstring. * device-x.c (Qmake_device_early_x_entry_point, Qmake_device_late_x_entry_point): New. Rename Qinit_pre_x_win, Qinit_post_x_win. (x_init_device): Call #'make-device-early-x-entry-point earlier, now we rely on it to find the application class and the app-defaults directory. (x_finish_init_device): Call #'make-device-late-x-entry-point with the created device. (Vx_app_defaults_directory): Always make this available, to simplify code in x-init.el. * device-tty.c (Qmake_device_early_tty_entry_point): New. Rename Qinit_pre_tty_win, rename Qinit_post_tty_win and move to frame-tty.c as Qmake_frame_after_init_entry_point. (tty_init_device): Call #'make-device-early-tty-entry-point before doing anything. * frame-tty.c (Qmake_frame_after_init_entry_point): New. * frame-tty.c (tty_after_init_frame): Have it call the better-named #'make-frame-after-init-entry-point function instead of #'init-post-tty-win (since it's called after frame, not device, creation). * device-msw.c (Qmake_device_early_mswindows_entry_point, Qmake_device_late_mswindows_entry_point): New. Rename Qinit_pre_mswindows_win, Qinit_post_mswindows_win. (mswindows_init_device): Call #'make-device-early-mswindows-entry-point here, instead of having its predecessor call us. (mswindows_finish_init_device): Call #'make-device-early-mswindows-entry-point, for symmetry with the other device types (though it's an empty function). * device-gtk.c (Qmake_device_early_gtk_entry_point, Qmake_device_late_gtk_entry_point): New. Rename Qinit_pre_gtk_win, Qinit_post_gtk_win. (gtk_init_device): Call #'make-device-early-gtk-entry-point; don't load ~/.xemacs/gtk-options.el ourselves, leave that to lisp. (gtk_finish_init_device): Call #'make-device-late-gtk-entry-point with the created device as an argument.
author Aidan Kehoe <kehoea@parhasard.net>
date Wed, 09 Jul 2008 20:46:22 +0200
parents 229bd619740a
children b3ce27ca7647 19a72041c5ed e0db3c197671
comparison
equal deleted inserted replaced
4475:86283c809984 4477:e34711681f30
54 54
55 #ifdef HAVE_BONOBO 55 #ifdef HAVE_BONOBO
56 #include <bonobo.h> 56 #include <bonobo.h>
57 #endif 57 #endif
58 58
59 /* Qdisplay in general.c */ 59 Lisp_Object Qmake_device_early_gtk_entry_point,
60 Lisp_Object Qinit_pre_gtk_win, Qinit_post_gtk_win; 60 Qmake_device_late_gtk_entry_point;
61 61
62 /* The application class of Emacs. */ 62 /* The application class of Emacs. */
63 Lisp_Object Vgtk_emacs_application_class; 63 Lisp_Object Vgtk_emacs_application_class;
64 64
65 Lisp_Object Vgtk_initial_argv_list; /* #### ugh! */ 65 Lisp_Object Vgtk_initial_argv_list; /* #### ugh! */
218 Lisp_Object display; 218 Lisp_Object display;
219 GtkWidget *app_shell = NULL; 219 GtkWidget *app_shell = NULL;
220 GdkVisual *visual = NULL; 220 GdkVisual *visual = NULL;
221 GdkColormap *cmap = NULL; 221 GdkColormap *cmap = NULL;
222 222
223 /* Run the early elisp side of the GTK device initialization. */
224 call0 (Qmake_device_early_gtk_entry_point);
225
223 /* gtk_init() and even gtk_check_init() are so brain dead that 226 /* gtk_init() and even gtk_check_init() are so brain dead that
224 getting an empty argv array causes them to abort. */ 227 getting an empty argv array causes them to abort. */
225 if (NILP (Vgtk_initial_argv_list)) 228 if (NILP (Vgtk_initial_argv_list))
226 { 229 {
227 invalid_operation ("gtk-initial-argv-list must be set before creating Gtk devices", Vgtk_initial_argv_list); 230 invalid_operation ("gtk-initial-argv-list must be set before creating Gtk devices", Vgtk_initial_argv_list);
264 Fgtk_init (Vgtk_initial_argv_list); 267 Fgtk_init (Vgtk_initial_argv_list);
265 268
266 #ifdef __FreeBSD__ 269 #ifdef __FreeBSD__
267 gdk_set_use_xshm (FALSE); 270 gdk_set_use_xshm (FALSE);
268 #endif 271 #endif
269
270 /* We attempt to load this file so that the user can set
271 ** gtk-initial-geometry and not need GNOME & session management to
272 ** set their default frame size. It also avoids the flicker
273 ** associated with setting the frame size in your .emacs file.
274 */
275 call4 (Qload, build_string ("~/.xemacs/gtk-options.el"), Qt, Qt, Qt);
276 272
277 #ifdef HAVE_GDK_IMLIB_INIT 273 #ifdef HAVE_GDK_IMLIB_INIT
278 /* Some themes in Gtk are so lame (most notably the Pixmap theme) 274 /* Some themes in Gtk are so lame (most notably the Pixmap theme)
279 that they rely on gdk_imlib, but don't call its initialization 275 that they rely on gdk_imlib, but don't call its initialization
280 routines. This makes them USELESS for non-gnome applications. 276 routines. This makes them USELESS for non-gnome applications.
334 330
335 DEVICE_GTK_GC_CACHE (d) = make_gc_cache (GTK_WIDGET (app_shell)); 331 DEVICE_GTK_GC_CACHE (d) = make_gc_cache (GTK_WIDGET (app_shell));
336 DEVICE_GTK_GRAY_PIXMAP (d) = NULL; 332 DEVICE_GTK_GRAY_PIXMAP (d) = NULL;
337 333
338 gtk_init_device_class (d); 334 gtk_init_device_class (d);
339
340 /* Run the elisp side of the X device initialization. */
341 call0 (Qinit_pre_gtk_win);
342 } 335 }
343 336
344 static void 337 static void
345 gtk_finish_init_device (struct device *UNUSED (d), Lisp_Object UNUSED (props)) 338 gtk_finish_init_device (struct device *d, Lisp_Object UNUSED (props))
346 { 339 {
347 call0 (Qinit_post_gtk_win); 340 call1 (Qmake_device_late_gtk_entry_point, wrap_device(d));
348 } 341 }
349 342
350 static void 343 static void
351 gtk_mark_device (struct device *d) 344 gtk_mark_device (struct device *d)
352 { 345 {
707 DEFSUBR (Fgtk_ungrab_pointer); 700 DEFSUBR (Fgtk_ungrab_pointer);
708 DEFSUBR (Fgtk_grab_keyboard); 701 DEFSUBR (Fgtk_grab_keyboard);
709 DEFSUBR (Fgtk_ungrab_keyboard); 702 DEFSUBR (Fgtk_ungrab_keyboard);
710 DEFSUBR (Fgtk_init); 703 DEFSUBR (Fgtk_init);
711 704
712 DEFSYMBOL (Qinit_pre_gtk_win); 705 DEFSYMBOL (Qmake_device_early_gtk_entry_point);
713 DEFSYMBOL (Qinit_post_gtk_win); 706 DEFSYMBOL (Qmake_device_late_gtk_entry_point);
714 } 707 }
715 708
716 void 709 void
717 console_type_create_device_gtk (void) 710 console_type_create_device_gtk (void)
718 { 711 {