Avoid unnecessary lock on UI thread with no model listeners eclipse-x…

…text#3251 Avoid unnecessarily aquiring a lock on the UI thread if there are no model listeners in the Xtext document. Signed-off-by: Martin Jobst <[email protected]>
mx990 · Nov 25, 2024 · 8a80d14 · 8a80d14
1 parent 574f7c3
commit 8a80d14
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/org.eclipse.xtext.ui/src/org/eclipse/xtext/ui/editor/model/XtextDocument.java b/org.eclipse.xtext.ui/src/org/eclipse/xtext/ui/editor/model/XtextDocument.java
@@ -563,6 +563,11 @@ protected <T> T internalReadOnly(IUnitOfWork<T, XtextResource> work, boolean isC
 		 *       changing while they run. To achieve this, we run the {@link IXtextModelListener}s on the UI thread.
 		 */
 		private void notifyModelListenersOnUiThread() {
+			synchronized (modelListeners) {
+				if (modelListeners.isEmpty()) {
+					return;
+				}
+			}
 			Display display = PlatformUI.getWorkbench().getDisplay();
 			if (Thread.currentThread() == display.getThread()) {
 				// We are already running on the display thread.  Run the listeners immediately.