summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlan Third <alan@idiocy.org>2021-05-22 10:26:58 +0100
committerAlan Third <alan@idiocy.org>2021-05-22 10:31:35 +0100
commit6ebbd6d38b65eafc19bfc178af216d3e96e29302 (patch)
tree447b6d417302555c5757f319c8b5ccfec3fb5b71
parent30e5d93ee1b938c207c4f5ec015ba2633cf423cd (diff)
downloademacs-scratch/ns/emacs27-drawing.tar.gz
Fix "flickering" on NSscratch/ns/emacs27-drawing
* 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.
-rw-r--r--src/nsterm.m33
1 files changed, 28 insertions, 5 deletions
diff --git a/src/nsterm.m b/src/nsterm.m
index 26cc9486141..f6e16035ea5 100644
--- a/src/nsterm.m
+++ b/src/nsterm.m
@@ -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 ();