[xemacs-hg @ 2003-02-09 09:33:42 by ben] walk-windows, redisplay fixes console-stream.c: Abort when any attempts to output a stream console are made. Should be caught sooner. event-msw.c: Don't redisplay() during sizing when the frame has not yet been initialized completely. event-stream.c, menubar-msw.c, menubar-x.c, menubar.c, menubar.h: Restore in_menu_callback. Bind it in menubar-{msw,x}.c when calling filter functions and the like. Conditionalize on it, not in_modal_loop, when issuing error in `next-event', otherwise we bite the dust immediately -- event-msw.c purposely calls Fnext_event() in a modal loop, and knows what it's doing. redisplay-output.c: Formatting fixes. redisplay.c, window.c, winslots.h: Delete lots of carcasses of attempts to add redisplay support for font-lock -- `pre/post-redisplay-hook', unimplemented junk from FSF (redisplay-end-trigger, `window-scroll-functions', `window-size-change-functions'). If we want to port some redisplay support from FSF, port the `fontified' property. redisplay.c: Put in a check here (as well as redisplay_device()) for a stream frame. We can get here directly through Lisp fun `redisplay-frame'. Abort if frame not initialized. redisplay.c: Check for stream frames/devices. window.el: walk-windows was broken when a frame was given to WHICH-FRAMES. it would loop forever. The FSF version fixes this but i didn't sync to them because (a) it conses (bad for lazy-lock), (b) it calls select-window.

--- a/src/event-stream.c	Sun Feb 09 07:03:07 2003 +0000
+++ b/src/event-stream.c	Sun Feb 09 09:33:48 2003 +0000
@@ -2243,9 +2243,13 @@
      for a timeout event, which will still get retrieved in such a case.
      However, the code to error in such a case has already been present for
      a long time without obvious problems so leaving it in isn't so
-     bad. --ben */
-  if (in_modal_loop)
-    invalid_operation ("Attempt to call next-event inside modal loop",
+     bad.
+
+     #### I used to conditionalize on in_modal_loop but that fails utterly
+     because event-msw.c specifically calls Fnext_event() inside of a modal
+     loop to clear the dispatch queue. --ben */
+  if (in_menu_callback)
+    invalid_operation ("Attempt to call next-event inside menu callback",
 		       Qunbound);
 
   depth = begin_dont_check_for_quit ();