tmf: Make TmfTimePreferences completely static
authorAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Tue, 15 Apr 2014 22:41:39 +0000 (18:41 -0400)
committerAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Mon, 4 Aug 2014 21:07:29 +0000 (17:07 -0400)
Marking the class final shows us that all the methods can be marked
as static, since they do not use any internal fields. This means
the getInstance() itself is unneeded.

Change-Id: I9fedc2804e30769f5f0b91650b0bc670df7181cc
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/25087
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
org.eclipse.linuxtools.tmf.core.tests/src/org/eclipse/linuxtools/tmf/core/tests/event/TmfTimePreferencesTest.java
org.eclipse.linuxtools.tmf.core.tests/src/org/eclipse/linuxtools/tmf/core/tests/event/TmfTimestampFormatTest.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/timestamp/TmfTimePreferences.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/timestamp/TmfTimestampFormat.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/properties/TmfTimestampFormatPage.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/TimeGraphScale.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/Utils.java

index 141bcf99518b347541069a2200bcff4921619942..0af3a0eb48bce8bba47c4dbc473109e5f2b3754e 100644 (file)
@@ -13,7 +13,6 @@
 package org.eclipse.linuxtools.tmf.core.tests.event;
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
 
 import java.util.Map;
 import java.util.TimeZone;
@@ -43,20 +42,12 @@ public class TmfTimePreferencesTest {
         assertEquals(DefaultScope.INSTANCE.getNode(Activator.PLUGIN_ID).get(ITmfTimePreferencesConstants.DATIME, null), ITmfTimePreferencesConstants.TIME_HOUR_FMT);
     }
 
-    /**
-     * Test that getInstance returns an instance
-     */
-    @Test
-    public void testGetInstance() {
-        assertNotNull(TmfTimePreferences.getInstance());
-    }
-
     /**
      * Test that getTimePattern returns the appropriate time pattern (from the default)
      */
     @Test
     public void testGetTimePattern() {
-        assertEquals(TIME_PATTERN, TmfTimePreferences.getInstance().getTimePattern());
+        assertEquals(TIME_PATTERN, TmfTimePreferences.getTimePattern());
     }
 
     /**
@@ -64,7 +55,7 @@ public class TmfTimePreferencesTest {
      */
     @Test
     public void testGetIntervalPattern() {
-        assertEquals(INTERVAL_PATTERN, TmfTimePreferences.getInstance().getIntervalPattern());
+        assertEquals(INTERVAL_PATTERN, TmfTimePreferences.getIntervalPattern());
     }
 
     /**
@@ -72,7 +63,7 @@ public class TmfTimePreferencesTest {
      */
     @Test
     public void testGetTimeZone() {
-        assertEquals(TimeZone.getDefault(), TmfTimePreferences.getInstance().getTimeZone());
+        assertEquals(TimeZone.getDefault(), TmfTimePreferences.getTimeZone());
     }
 
     /**
@@ -81,7 +72,7 @@ public class TmfTimePreferencesTest {
      */
     @Test
     public void testGetPreferenceMap() {
-        Map<String, String> defaultPreferenceMap = TmfTimePreferences.getInstance().getDefaultPreferenceMap();
+        Map<String, String> defaultPreferenceMap = TmfTimePreferences.getDefaultPreferenceMap();
         assertEquals(ITmfTimePreferencesConstants.TIME_HOUR_FMT, defaultPreferenceMap.get(ITmfTimePreferencesConstants.DATIME));
 
         // Modify the preferences
@@ -93,11 +84,11 @@ public class TmfTimePreferencesTest {
         } catch (BackingStoreException e) {
         }
         // Make sure the modification is in the map
-        Map<String, String> preferenceMap = TmfTimePreferences.getInstance().getPreferenceMap();
+        Map<String, String> preferenceMap = TmfTimePreferences.getPreferenceMap();
         assertEquals(testValue, preferenceMap.get(ITmfTimePreferencesConstants.DATIME));
 
         // Make sure the default is still the same
-        defaultPreferenceMap = TmfTimePreferences.getInstance().getDefaultPreferenceMap();
+        defaultPreferenceMap = TmfTimePreferences.getDefaultPreferenceMap();
         assertEquals(ITmfTimePreferencesConstants.TIME_HOUR_FMT, defaultPreferenceMap.get(ITmfTimePreferencesConstants.DATIME));
     }
 
@@ -106,7 +97,7 @@ public class TmfTimePreferencesTest {
      */
     @Test
     public void testComputeTimePattern() {
-        assertEquals(TIME_PATTERN, TmfTimePreferences.getInstance().computeTimePattern(TmfTimePreferences.getInstance().getPreferenceMap()));
+        assertEquals(TIME_PATTERN, TmfTimePreferences.computeTimePattern(TmfTimePreferences.getPreferenceMap()));
     }
 
 }
index 523c697ab2ff6bdf0948bbfd6e6b9447b33c5cf8..a967d449aba7f9c796520f83df69a79cd7f17317 100644 (file)
@@ -106,7 +106,7 @@ public class TmfTimestampFormatTest {
      */
     @Test
     public void testGetDefaulTimeFormat() {
-        assertEquals(TmfTimestampFormat.getDefaulTimeFormat().toPattern(), TmfTimePreferences.getInstance().getTimePattern());
+        assertEquals(TmfTimestampFormat.getDefaulTimeFormat().toPattern(), TmfTimePreferences.getTimePattern());
     }
 
     /**
@@ -115,7 +115,7 @@ public class TmfTimestampFormatTest {
      */
     @Test
     public void testGetDefaulIntervalFormat() {
-        assertEquals(TmfTimestampFormat.getDefaulIntervalFormat().toPattern(), TmfTimePreferences.getInstance().getIntervalPattern());
+        assertEquals(TmfTimestampFormat.getDefaulIntervalFormat().toPattern(), TmfTimePreferences.getIntervalPattern());
     }
 
     /**
index be8e17d31fdd8e3e70abcf0c2de924624c00aff7..f1dcfbf394695cca3b9d325b758652f836477015 100644 (file)
@@ -30,7 +30,7 @@ import org.eclipse.linuxtools.internal.tmf.core.Activator;
  * @version 1.0
  * @since 2.1
  */
-public class TmfTimePreferences {
+public final class TmfTimePreferences {
 
     // ------------------------------------------------------------------------
     // Constants
@@ -42,11 +42,6 @@ public class TmfTimePreferences {
     private static final String TIME_DELIMITER_DEFAULT = ITmfTimePreferencesConstants.DELIMITER_COLON;
     private static final String SSEC_DELIMITER_DEFAULT = ITmfTimePreferencesConstants.DELIMITER_SPACE;
     private static final String TIME_ZONE_DEFAULT = TimeZone.getDefault().getID();
-    // ------------------------------------------------------------------------
-    // Attributes
-    // ------------------------------------------------------------------------
-
-    private static TmfTimePreferences fPreferences;
 
     // ------------------------------------------------------------------------
     // Constructor
@@ -64,21 +59,7 @@ public class TmfTimePreferences {
         defaultPreferences.put(ITmfTimePreferencesConstants.SSEC_DELIMITER, SSEC_DELIMITER_DEFAULT);
         defaultPreferences.put(ITmfTimePreferencesConstants.TIME_ZONE, TIME_ZONE_DEFAULT);
 
-        // Create the singleton and update default formats
-        getInstance();
-    }
-
-    /**
-     * Get the TmfTimePreferences singleton
-     *
-     * @return The TmfTimePreferences instance
-     */
-    public static synchronized TmfTimePreferences getInstance() {
-        if (fPreferences == null) {
-            fPreferences = new TmfTimePreferences();
-            TmfTimestampFormat.updateDefaultFormats();
-        }
-        return fPreferences;
+        TmfTimestampFormat.updateDefaultFormats();
     }
 
     /**
@@ -96,7 +77,7 @@ public class TmfTimePreferences {
      *
      * @return the timestamp pattern
      */
-    public String getTimePattern() {
+    public static String getTimePattern() {
         return computeTimePattern(getPreferenceMap(false));
     }
 
@@ -105,7 +86,7 @@ public class TmfTimePreferences {
      *
      * @return the interval pattern
      */
-    public String getIntervalPattern() {
+    public static String getIntervalPattern() {
         return computeIntervalPattern(getPreferenceMap(false));
     }
 
@@ -114,7 +95,7 @@ public class TmfTimePreferences {
      *
      * @return the time zone
      */
-    public TimeZone getTimeZone() {
+    public static TimeZone getTimeZone() {
         return TimeZone.getTimeZone(Platform.getPreferencesService().getString(Activator.PLUGIN_ID, ITmfTimePreferencesConstants.TIME_ZONE, TimeZone.getDefault().getID(), null));
     }
 
@@ -123,7 +104,7 @@ public class TmfTimePreferences {
      *
      * @return a collection containing the default preferences
      */
-    public Map<String, String> getDefaultPreferenceMap() {
+    public static Map<String, String> getDefaultPreferenceMap() {
         return getPreferenceMap(true);
     }
 
@@ -132,7 +113,7 @@ public class TmfTimePreferences {
      *
      * @return a collection containing the current preferences
      */
-    public Map<String, String> getPreferenceMap() {
+    public static Map<String, String> getPreferenceMap() {
         return getPreferenceMap(false);
     }
 
@@ -173,7 +154,7 @@ public class TmfTimePreferences {
      * @param prefsMap the preferences to apply when computing the time pattern
      * @return the time pattern resulting in applying the preferences
      */
-    public String computeTimePattern(Map<String, String> prefsMap) {
+    public static String computeTimePattern(Map<String, String> prefsMap) {
         String dateTimeFormat = prefsMap.get(ITmfTimePreferencesConstants.DATIME);
         if (dateTimeFormat == null) {
             dateTimeFormat = ITmfTimePreferencesConstants.DEFAULT_TIME_PATTERN;
index 3259c150e83e292ba22ed469f7a4f29313abe949..60f5c7dc303f0acbdebb99bb026b486bfdb70f15 100644 (file)
@@ -244,7 +244,7 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      * The default constructor (uses the default pattern)
      */
     public TmfTimestampFormat() {
-        this(TmfTimePreferences.getInstance().getTimePattern());
+        this(TmfTimePreferences.getTimePattern());
     }
 
     /**
@@ -285,8 +285,8 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      * @since 2.1
      */
     public static void updateDefaultFormats() {
-        fDefaultTimeFormat = new TmfTimestampFormat(TmfTimePreferences.getInstance().getTimePattern(), TmfTimePreferences.getInstance().getTimeZone());
-        fDefaultIntervalFormat = new TmfTimestampFormat(TmfTimePreferences.getInstance().getIntervalPattern());
+        fDefaultTimeFormat = new TmfTimestampFormat(TmfTimePreferences.getTimePattern(), TmfTimePreferences.getTimeZone());
+        fDefaultIntervalFormat = new TmfTimestampFormat(TmfTimePreferences.getIntervalPattern());
     }
 
     /**
@@ -294,7 +294,7 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      */
     public static TmfTimestampFormat getDefaulTimeFormat() {
         if (fDefaultTimeFormat == null) {
-            fDefaultTimeFormat = new TmfTimestampFormat(TmfTimePreferences.getInstance().getTimePattern(), TmfTimePreferences.getInstance().getTimeZone());
+            fDefaultTimeFormat = new TmfTimestampFormat(TmfTimePreferences.getTimePattern(), TmfTimePreferences.getTimeZone());
         }
         return fDefaultTimeFormat;
     }
@@ -304,7 +304,7 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      */
     public static TmfTimestampFormat getDefaulIntervalFormat() {
         if (fDefaultIntervalFormat == null) {
-            fDefaultIntervalFormat = new TmfTimestampFormat(TmfTimePreferences.getInstance().getIntervalPattern());
+            fDefaultIntervalFormat = new TmfTimestampFormat(TmfTimePreferences.getIntervalPattern());
         }
         return fDefaultIntervalFormat;
     }
index 5c37a23eb2c5a79cd3df0c60c671ae9e7326b338..be5371b37e4e6c62761c16a2d5d6441e0baf06e0 100644 (file)
@@ -153,7 +153,6 @@ public class TmfTimestampFormatPage extends PreferencePage implements IWorkbench
     // General stuff
     private Composite fPage;
     private IPreferenceStore fPreferenceStore;
-    private TmfTimePreferences fTimePreference;
 
     // Example section
     private Composite fExampleSection;
@@ -187,8 +186,7 @@ public class TmfTimestampFormatPage extends PreferencePage implements IWorkbench
      */
     public TmfTimestampFormatPage() {
         fPreferenceStore = getPreferenceStore();
-        fTimePreference = TmfTimePreferences.getInstance();
-        fPreferenceMap = fTimePreference.getPreferenceMap();
+        fPreferenceMap = TmfTimePreferences.getPreferenceMap();
     }
 
     // ------------------------------------------------------------------------
@@ -306,7 +304,7 @@ public class TmfTimestampFormatPage extends PreferencePage implements IWorkbench
         fSSecFieldDelim.loadDefault();
         fCombo.loadDefault();
 
-        fPreferenceMap = TmfTimePreferences.getInstance().getDefaultPreferenceMap();
+        fPreferenceMap = TmfTimePreferences.getDefaultPreferenceMap();
         displayExample();
     }
 
@@ -380,7 +378,7 @@ public class TmfTimestampFormatPage extends PreferencePage implements IWorkbench
 
     private void displayExample() {
         long ts = 1332170682500677380L;
-        String timePattern = fTimePreference.computeTimePattern(fPreferenceMap);
+        String timePattern = TmfTimePreferences.computeTimePattern(fPreferenceMap);
         fPatternDisplay.setText(timePattern);
         fPatternDisplay.redraw();
 
index 1dda20588f8b8e8eef1de11dffc504fe2dfbb8fa..2128e01659fc0ee7effa191c71033850effbfe13 100644 (file)
@@ -588,7 +588,7 @@ abstract class TimeDraw {
      * Updates the timezone using the preferences.
      */
     public static void updateTimeZone() {
-        final TimeZone timeZone = TmfTimePreferences.getInstance().getTimeZone();
+        final TimeZone timeZone = TmfTimePreferences.getTimeZone();
         for (SimpleDateFormat sdf : formatArray) {
             sdf.setTimeZone(timeZone);
         }
index a75c5d9fe3ccb0dd3c5c857d3be5208e51a85d7d..0440f488fdbc04d0879299593f476c8dc0a23667 100644 (file)
@@ -89,7 +89,7 @@ public class Utils {
      * @since 2.1
      */
     public static void updateTimeZone() {
-        TimeZone timeZone = TmfTimePreferences.getInstance().getTimeZone();
+        TimeZone timeZone = TmfTimePreferences.getTimeZone();
         TIME_FORMAT.setTimeZone(timeZone);
         DATE_FORMAT.setTimeZone(timeZone);
     }
This page took 0.032766 seconds and 5 git commands to generate.