]> git.eshelyaron.com Git - emacs.git/commitdiff
Fix "flickering" on NS scratch/ns/emacs27-drawing
authorAlan Third <alan@idiocy.org>
Sat, 22 May 2021 09:26:58 +0000 (10:26 +0100)
committerAlan Third <alan@idiocy.org>
Sat, 22 May 2021 09:31:35 +0000 (10:31 +0100)
* src/nsterm.m ([EmacsView initFrameFromEmacs:]): Add some settings
that allow the frame to redraw properly while resizing.
([EmacsView viewWillDraw]): Call redisplay if we think expose_frame is
going to refuse to draw anything.
([EmacsView drawRect:]): There's no point trying to draw anything if
the frame is still garbaged.

src/nsterm.m

index 26cc948614115edacd566524d4dfa7fa2632f263..f6e16035ea5f90311774a26276fa5a46f088fc28 100644 (file)
@@ -7454,6 +7454,15 @@ not_in_argv (NSString *arg)
   [self initWithFrame: r];
   [self setAutoresizingMask: NSViewWidthSizable | NSViewHeightSizable];
 
+    /* These settings mean AppKit will retain the contents of the frame
+     on resize.  Unfortunately it also means the frame will not be
+     automatically marked for display, but we can do that ourselves in
+     viewDidResize.  */
+  [self setWantsLayer:YES];
+  [self setLayerContentsRedrawPolicy:
+          NSViewLayerContentsRedrawOnSetNeedsDisplay];
+  [self setLayerContentsPlacement:NSViewLayerContentsPlacementTopLeft];
+
   FRAME_NS_VIEW (f) = self;
   emacsframe = f;
 #ifdef NS_IMPL_COCOA
@@ -8254,10 +8263,23 @@ not_in_argv (NSString *arg)
 
 - (void)viewWillDraw
 {
-  /* If the frame has been garbaged there's no point in redrawing
-     anything.  */
-  if (FRAME_GARBAGED_P (emacsframe))
-    [self setNeedsDisplay:NO];
+  if (FRAME_GARBAGED_P (emacsframe)
+      && !redisplaying_p)
+    {
+      /* If there is IO going on when redisplay is run here Emacs
+         crashes.  I think it's because this code will always be run
+         within the run loop and for whatever reason processing input
+         is dangerous.  This technique was stolen wholesale from
+         nsmenu.m and seems to work.  */
+      bool owfi = waiting_for_input;
+      waiting_for_input = 0;
+      block_input ();
+
+      redisplay ();
+
+      unblock_input ();
+      waiting_for_input = owfi;
+    }
 }
 
 - (void)drawRect: (NSRect)rect
@@ -8268,7 +8290,8 @@ not_in_argv (NSString *arg)
   NSTRACE ("[EmacsView drawRect:" NSTRACE_FMT_RECT "]",
            NSTRACE_ARG_RECT(rect));
 
-  if (!emacsframe || !emacsframe->output_data.ns)
+  if (!emacsframe || !emacsframe->output_data.ns
+      || FRAME_GARBAGED_P (emacsframe))
     return;
 
   block_input ();