Cleanup (daemon test to use existing API instead of impl. method; formatting; typos)
authorRoman Shevchenko <roman.shevchenko@jetbrains.com>
Tue, 24 Feb 2015 18:57:54 +0000 (19:57 +0100)
committerRoman Shevchenko <roman.shevchenko@jetbrains.com>
Tue, 24 Feb 2015 18:57:54 +0000 (19:57 +0100)
platform/platform-api/src/com/intellij/ide/GeneralSettings.java
platform/platform-impl/src/com/intellij/openapi/application/impl/ApplicationImpl.java

index 64da1496616c5dad8ed30f2262f4c070e649645a..16cf8e14605517cd60ebfc91f13346c68e5f51b5 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2014 JetBrains s.r.o.
+ * Copyright 2000-2015 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,7 +22,6 @@ import com.intellij.util.xmlb.XmlSerializerUtil;
 import com.intellij.util.xmlb.annotations.OptionTag;
 import com.intellij.util.xmlb.annotations.Transient;
 import org.intellij.lang.annotations.MagicConstant;
-import org.jetbrains.annotations.NonNls;
 import org.jetbrains.annotations.Nullable;
 
 import java.beans.PropertyChangeListener;
@@ -37,7 +36,7 @@ public class GeneralSettings implements PersistentStateComponent<GeneralSettings
   public static final int OPEN_PROJECT_NEW_WINDOW = 0;
   public static final int OPEN_PROJECT_SAME_WINDOW = 1;
 
-  @NonNls public static final String PROP_INACTIVE_TIMEOUT = "inactiveTimeout";
+  public static final String PROP_INACTIVE_TIMEOUT = "inactiveTimeout";
 
   private String myBrowserPath = BrowserUtil.getDefaultAlternativeBrowserPath();
   private boolean myShowTipsOnStartup = true;
index 0d2c973eb4a2098b96ae6a996ae32fdcfdfdcce5..d9f96559395830c56f0d141895044e309a6e43aa 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2014 JetBrains s.r.o.
+ * Copyright 2000-2015 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -1352,8 +1352,10 @@ public class ApplicationImpl extends PlatformComponentManagerImpl implements App
     myDispatcher.getMulticaster().writeActionFinished(action);
   }
 
-  // public for testing purposes
-  public void _saveSettings() {
+  @Override
+  public void saveSettings() {
+    if (myDoNotSave) return;
+
     if (mySaveSettingsIsInProgress.compareAndSet(false, true)) {
       try {
         StoreUtil.save(getStateStore(), null);
@@ -1364,12 +1366,6 @@ public class ApplicationImpl extends PlatformComponentManagerImpl implements App
     }
   }
 
-  @Override
-  public void saveSettings() {
-    if (myDoNotSave) return;
-    _saveSettings();
-  }
-
   @Override
   public void saveAll() {
     if (myDoNotSave) return;