refactor(mps-sync-plugin): use Mutex instead of Semaphore(1)

modelix · Apr 22, 2024 · fefec27 · fefec27
1 parent 1fdcf25
commit fefec27
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/mps-sync-plugin/src/main/kotlin/org/modelix/mps/sync/plugin/gui/ModelSyncGuiFactory.kt b/mps-sync-plugin/src/main/kotlin/org/modelix/mps/sync/plugin/gui/ModelSyncGuiFactory.kt
@@ -29,7 +29,7 @@ import com.intellij.ui.content.ContentFactory
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
-import kotlinx.coroutines.sync.Semaphore
+import kotlinx.coroutines.sync.Mutex
 import mu.KotlinLogging
 import org.modelix.kotlin.utils.UnstableModelixFeature
 import org.modelix.model.api.BuiltinLanguages
@@ -88,7 +88,7 @@ class ModelSyncGuiFactory : ToolWindowFactory, Disposable {
 
         private val logger = KotlinLogging.logger {}
         private val coroutineScope = CoroutineScope(Dispatchers.Default)
-        private val mutex = Semaphore(1)
+        private val mutex = Mutex()
 
         val contentPanel = JPanel()
         val bindingsRefresher: BindingsComboBoxRefresher
@@ -346,15 +346,15 @@ class ModelSyncGuiFactory : ToolWindowFactory, Disposable {
 
         private fun callOnlyIfNotFetching(action: suspend () -> Unit) {
             coroutineScope.launch {
-                if (mutex.tryAcquire()) {
+                if (mutex.tryLock()) {
                     try {
                         setUiControlsEnabled(false)
                         action()
                     } catch (ex: Exception) {
                         logger.error(ex) { "Failed to fetch data from server." }
                     } finally {
                         setUiControlsEnabled(true)
-                        mutex.release()
+                        mutex.unlock()
                     }
                 }
             }