[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#23924: [PATCH] Post AppDefined events from the main thread ONLY (bug
From: |
Alan Third |
Subject: |
bug#23924: [PATCH] Post AppDefined events from the main thread ONLY (bug#23934) |
Date: |
Wed, 20 Jul 2016 22:09:26 +0100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
* src/nsterm.h: Make nextappdefined var not just GNUStep.
* src/nsterm.c (ns_send_appdefined, sendFromMainThread): Remove GNUStep
---
src/nsterm.h | 2 +-
src/nsterm.m | 5 +----
2 files changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/nsterm.h b/src/nsterm.h
index 862ff2e..3d8b1a1 100644
--- a/src/nsterm.h
+++ b/src/nsterm.h
@@ -380,9 +380,9 @@ char const * nstrace_fullscreen_type_name (int);
#endif
#ifdef NS_IMPL_GNUSTEP
BOOL applicationDidFinishLaunchingCalled;
+#endif
@public
int nextappdefined;
-#endif
}
- (void)logNotification: (NSNotification *)notification;
- (void)antialiasThresholdDidChange:(NSNotification *)notification;
diff --git a/src/nsterm.m b/src/nsterm.m
index 8da2ffe..dcc1e87 100644
--- a/src/nsterm.m
+++ b/src/nsterm.m
@@ -3927,8 +3927,8 @@ overwriting cursor (usually when cursor on a tab) */
{
NSTRACE_WHEN (NSTRACE_GROUP_EVENTS, "ns_send_appdefined(%d)", value);
-#ifdef NS_IMPL_GNUSTEP
// GNUstep needs postEvent to happen on the main thread.
+ // Cocoa needs nextEventMatchingMask to happen on the main thread too.
if (! [[NSThread currentThread] isMainThread])
{
EmacsApp *app = (EmacsApp *)NSApp;
@@ -3938,7 +3938,6 @@ overwriting cursor (usually when cursor on a tab) */
waitUntilDone:YES];
return;
}
-#endif
/* Only post this event if we haven't already posted one. This will end
the [NXApp run] main loop after having processed all events queued at
@@ -5551,12 +5550,10 @@ - (void)timeout_handler: (NSTimer *)timedEntry
ns_send_appdefined (-2);
}
-#ifdef NS_IMPL_GNUSTEP
- (void)sendFromMainThread:(id)unused
{
ns_send_appdefined (nextappdefined);
}
-#endif
- (void)fd_handler:(id)unused
/* --------------------------------------------------------------------------
--
Hi Bob, can you try this patch instead.
Rather than remove the check, I’m now trying to force it to run in the
main thread, since that’s what your original error was complaining
about.
I’ve tried looking for more information on this bug in 10.10, and am
drawing a blank. I’m beginning to suspect it’s a bug in Emacs’s event
handling code rather than OS X.
--
Alan Third
- bug#23924: 25.1.50; Emacs NSInternalInconsistencyException on macos Sierra developer beta 2, (continued)
- bug#23924: 25.1.50; Emacs NSInternalInconsistencyException on macos Sierra developer beta 2, Alan Third, 2016/07/10
- bug#23924: 25.1.50; Emacs NSInternalInconsistencyException on macos Sierra developer beta 2, Bob Halley, 2016/07/10
- bug#23924: 25.1.50; Emacs NSInternalInconsistencyException on macos Sierra developer beta 2, Bob Halley, 2016/07/10
- bug#23924: 25.1.50; Emacs NSInternalInconsistencyException on macos Sierra developer beta 2, Bob Halley, 2016/07/11
- bug#23924: 25.1.50; Emacs NSInternalInconsistencyException on macos Sierra developer beta 2, Bob Halley, 2016/07/11
- bug#23924: [PATCH] Only check for AppDefined events in OS X 10.10 (bug#23924), Alan Third, 2016/07/13
- bug#23924: [PATCH] Only check for AppDefined events in OS X 10.10 (bug#23924), Bob Halley, 2016/07/13
- bug#23924: [PATCH] Only check for AppDefined events in OS X 10.10 (bug#23924), Alan Third, 2016/07/17
- bug#23924: [PATCH] Only check for AppDefined events in OS X 10.10 (bug#23924), Bob Halley, 2016/07/20
- bug#23924: [PATCH] Only check for AppDefined events in OS X 10.10 (bug#23924), Alan Third, 2016/07/20
- bug#23924: [PATCH] Post AppDefined events from the main thread ONLY (bug#23934),
Alan Third <=
- bug#23924: [PATCH] Post AppDefined events from the main thread ONLY (bug#23934), Bob Halley, 2016/07/21
- bug#23924: [PATCH] Post AppDefined events from the main thread ONLY (bug#23934), Bob Halley, 2016/07/22
- bug#23924: [PATCH] Post AppDefined events from the main thread ONLY (bug#23934), Alan Third, 2016/07/22