synchronize GroovyLogManager.manager operations #877

diffplug · Jun 15, 2021 · b123f17 · b123f17
1 parent 5e8c3b8
commit b123f17
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/.../src/main/java/com/diffplug/spotless/extra/eclipse/groovy/GrEclipseFormatterStepImpl.java b/.../src/main/java/com/diffplug/spotless/extra/eclipse/groovy/GrEclipseFormatterStepImpl.java
@@ -106,7 +106,9 @@ public GroovyErrorListener() {
 			errors = Collections.synchronizedList(new ArrayList<String>());
 			ILog groovyLogger = GroovyCoreActivator.getDefault().getLog();
 			groovyLogger.addLogListener(this);
-			GroovyLogManager.manager.addLogger(this);
+			synchronized(GroovyLogManager.manager) {
+				GroovyLogManager.manager.addLogger(this);
+			}
 		}
 
 		@Override
@@ -117,7 +119,9 @@ public void logging(final IStatus status, final String plugin) {
 		public boolean errorsDetected() {
 			ILog groovyLogger = GroovyCoreActivator.getDefault().getLog();
 			groovyLogger.removeLogListener(this);
-			GroovyLogManager.manager.removeLogger(this);
+			synchronized(GroovyLogManager.manager) {
+				GroovyLogManager.manager.removeLogger(this);
+			}
 			return 0 != errors.size();
 		}