David Reitter <[EMAIL PROTECTED]> writes:

> I've been having trouble with the old version of the patch when
> changing font settings.  If you change the font (e.g., with  M-x set- 
> frame-font, or with the font panel), the frame resizes, and it is not
> full-screen any longer.
>
> I haven't installed the new patch yet.  It would be interesting to see
> if things work better with that.

David,

I had noticed that problem, too (and my patch did not address it).  I
tried right now to call fullscreen_hook from x_set_font (see patch
below), and it seems to work.  I think it's a little ugly, however.
It would be better to intercept the lower-level code that actually
changes the size (in pixels) of the Mac window, but I couldn't find it
in the time I have available today.  What do you think?

Vebjorn


--- ../../old/emacs.orig/src/frame.c    2008-06-02 16:25:50.000000000 -0400
+++ frame.c     2008-06-09 14:57:20.000000000 -0400
@@ -3142,6 +3142,9 @@
   else
     abort ();
 
+  if (fullscreen_hook != NULL) 
+    fullscreen_hook (f);
+
   do_pending_window_change (0);
 
   /* Don't call `face-set-after-frame-default' when faces haven't been

--~--~---------~--~----~------------~-------~--~----~
Carbon Emacs User Group
http://groups.google.com/group/carbon-emacs?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to