tmf: Make TmfTimePreferences completely static
authorAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Wed, 22 Oct 2014 13:40:53 +0000 (09:40 -0400)
committerAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Thu, 23 Oct 2014 22:08:22 +0000 (18:08 -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: Iaed5876fd3f6151fceb90991d5deb452c6e70ef3
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/35334
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
org.eclipse.tracecompass.tmf.core.tests/src/org/eclipse/tracecompass/tmf/core/tests/event/TmfTimePreferencesTest.java
org.eclipse.tracecompass.tmf.core.tests/src/org/eclipse/tracecompass/tmf/core/tests/event/TmfTimestampFormatTest.java
org.eclipse.tracecompass.tmf.core.tests/stubs/org/eclipse/tracecompass/tmf/tests/stubs/trace/text/SyslogTrace.java
org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/timestamp/TmfTimePreferences.java
org.eclipse.tracecompass.tmf.core/src/org/eclipse/tracecompass/tmf/core/timestamp/TmfTimestampFormat.java
org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/properties/TmfTimestampFormatPage.java
org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/widgets/TimeGraphScale.java
org.eclipse.tracecompass.tmf.ui/src/org/eclipse/tracecompass/tmf/ui/widgets/timegraph/widgets/Utils.java

index 36d88595ace7e7b28011d4b15fbe939bbed2d92d..9268454093c6f4141a4bf6d1a44c95727c297249 100644 (file)
@@ -13,7 +13,6 @@
 package org.eclipse.tracecompass.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 fa95d83fd2f409243c7e8f6c76e12c68e8c05779..a4f22ac5104d171e0d9ccec6b20b4b1105f7dab6 100644 (file)
@@ -108,7 +108,7 @@ public class TmfTimestampFormatTest {
      */
     @Test
     public void testGetDefaulTimeFormat() {
-        assertEquals(TmfTimestampFormat.getDefaulTimeFormat().toPattern(), TmfTimePreferences.getInstance().getTimePattern());
+        assertEquals(TmfTimestampFormat.getDefaulTimeFormat().toPattern(), TmfTimePreferences.getTimePattern());
     }
 
     /**
@@ -117,7 +117,7 @@ public class TmfTimestampFormatTest {
      */
     @Test
     public void testGetDefaulIntervalFormat() {
-        assertEquals(TmfTimestampFormat.getDefaulIntervalFormat().toPattern(), TmfTimePreferences.getInstance().getIntervalPattern());
+        assertEquals(TmfTimestampFormat.getDefaulIntervalFormat().toPattern(), TmfTimePreferences.getIntervalPattern());
     }
 
     /**
index 8f3cd46608415d998258d3a8b6167f0694db50e4..6b4102f1d6064c7d8e5447c66647795eaf0c9819 100644 (file)
@@ -39,7 +39,7 @@ public class SyslogTrace extends TextTrace<SyslogEvent> {
     public static final String TIMESTAMP_FORMAT = "MMM dd HH:mm:ss"; //$NON-NLS-1$
     /** The corresponding date format of the time stamp. */
     public static final SimpleDateFormat TIMESTAMP_SIMPLEDATEFORMAT = new SimpleDateFormat(
-            TIMESTAMP_FORMAT, TmfTimePreferences.getInstance().getLocale());
+            TIMESTAMP_FORMAT, TmfTimePreferences.getLocale());
     /** The regular expression pattern of the first line of an event. */
     public static final Pattern LINE1_PATTERN = Pattern.compile(
             "\\s*(\\S\\S\\S \\d\\d? \\d\\d:\\d\\d:\\d\\d)\\s*(\\S*)\\s*(\\S*):+\\s*(.*\\S)?"); //$NON-NLS-1$
index 9248201f1dc124c11421f5c1da014645f071ae1e..6adf30582c52c100692d46090225aa452e88311b 100644 (file)
@@ -31,7 +31,7 @@ import org.eclipse.tracecompass.internal.tmf.core.Activator;
  * @version 1.0
  * @since 2.1
  */
-public class TmfTimePreferences {
+public final class TmfTimePreferences {
 
     // ------------------------------------------------------------------------
     // Constants
@@ -43,16 +43,17 @@ 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
     // ------------------------------------------------------------------------
 
+    /**
+     * Local constructor
+     */
+    private TmfTimePreferences() {
+    }
+
     /**
      * Initialize the default preferences and the singleton
      */
@@ -65,27 +66,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;
-    }
-
-    /**
-     * Local constructor
-     */
-    private TmfTimePreferences() {
+        TmfTimestampFormat.updateDefaultFormats();
     }
 
     // ------------------------------------------------------------------------
@@ -97,7 +78,7 @@ public class TmfTimePreferences {
      *
      * @return the timestamp pattern
      */
-    public String getTimePattern() {
+    public static String getTimePattern() {
         return computeTimePattern(getPreferenceMap(false));
     }
 
@@ -106,7 +87,7 @@ public class TmfTimePreferences {
      *
      * @return the interval pattern
      */
-    public String getIntervalPattern() {
+    public static String getIntervalPattern() {
         return computeIntervalPattern(getPreferenceMap(false));
     }
 
@@ -115,7 +96,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));
     }
 
@@ -125,7 +106,7 @@ public class TmfTimePreferences {
      * @return the locale
      * @since 3.2
      */
-    public Locale getLocale() {
+    public static Locale getLocale() {
         return Locale.forLanguageTag(Platform.getPreferencesService().getString(Activator.PLUGIN_ID, ITmfTimePreferencesConstants.LOCALE, Locale.getDefault().toLanguageTag(), null));
     }
 
@@ -134,7 +115,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);
     }
 
@@ -143,7 +124,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);
     }
 
@@ -184,7 +165,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 498d2c006b4b9c263223c4dd3701d3f89c3244e3..c0fda805ca233bff32b1ea8c730796b980e6aec0 100644 (file)
@@ -248,7 +248,7 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      * The default constructor (uses the default pattern)
      */
     public TmfTimestampFormat() {
-        this(TmfTimePreferences.getInstance().getTimePattern());
+        this(TmfTimePreferences.getTimePattern());
     }
 
     /**
@@ -308,10 +308,10 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      */
     public static void updateDefaultFormats() {
         fDefaultTimeFormat = new TmfTimestampFormat(
-                TmfTimePreferences.getInstance().getTimePattern(),
-                TmfTimePreferences.getInstance().getTimeZone(),
-                TmfTimePreferences.getInstance().getLocale());
-        fDefaultIntervalFormat = new TmfTimestampFormat(TmfTimePreferences.getInstance().getIntervalPattern());
+                TmfTimePreferences.getTimePattern(),
+                TmfTimePreferences.getTimeZone(),
+                TmfTimePreferences.getLocale());
+        fDefaultIntervalFormat = new TmfTimestampFormat(TmfTimePreferences.getIntervalPattern());
     }
 
     /**
@@ -320,9 +320,9 @@ public class TmfTimestampFormat extends SimpleDateFormat {
     public static TmfTimestampFormat getDefaulTimeFormat() {
         if (fDefaultTimeFormat == null) {
             fDefaultTimeFormat = new TmfTimestampFormat(
-                    TmfTimePreferences.getInstance().getTimePattern(),
-                    TmfTimePreferences.getInstance().getTimeZone(),
-                    TmfTimePreferences.getInstance().getLocale());
+                    TmfTimePreferences.getTimePattern(),
+                    TmfTimePreferences.getTimeZone(),
+                    TmfTimePreferences.getLocale());
         }
         return fDefaultTimeFormat;
     }
@@ -332,7 +332,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 bc0dfa5c0ccee23bccfe734abdcb1b87b7fb496b..a4eb53e18294c19f0f8626abb58afcbe821491ab 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 ce2218dbc0f2961af5d7436f2efa360e16b3b1f1..250750c145ef08e17b7e037d9e887221a14dc641 100644 (file)
@@ -606,7 +606,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 5f8f8a465273f92cdf2405c540e5784a4fd614bf..a36a06ae3f40251d1d712e60bf18b4cd79824eee 100644 (file)
@@ -100,7 +100,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.056942 seconds and 5 git commands to generate.