tmf: Fix remaining Java warnings in tmf.ui
authorAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Fri, 4 Jan 2013 20:28:12 +0000 (15:28 -0500)
committerAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Mon, 7 Jan 2013 23:16:57 +0000 (18:16 -0500)
Change-Id: I02cc52ad3dca97512c29aa7d92c2ce32005e8e73
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/9467
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Hudson CI
16 files changed:
org.eclipse.linuxtools.tmf.ui/.settings/.api_filters [new file with mode: 0644]
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/custom/CustomTxtTrace.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomTxtParserInputWizardPage.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomTxtParserOutputWizardPage.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomTxtParserWizard.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomXmlParserInputWizardPage.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomXmlParserOutputWizardPage.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/internal/tmf/ui/parsers/wizards/CustomXmlParserWizard.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/wizards/ImportTraceWizardPage.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/viewers/events/TmfEventsCache.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/viewers/events/TmfEventsTable.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/timechart/TimeChartView.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/TimeGraphViewer.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/ITimeDataProvider.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/TimeGraphControl.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/widgets/timegraph/widgets/TimeGraphScale.java

diff --git a/org.eclipse.linuxtools.tmf.ui/.settings/.api_filters b/org.eclipse.linuxtools.tmf.ui/.settings/.api_filters
new file mode 100644 (file)
index 0000000..ff31090
--- /dev/null
@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<component id="org.eclipse.linuxtools.tmf.ui" version="2">
+    <resource path="src/org/eclipse/linuxtools/tmf/ui/editors/UnsortedPropertySheetPage.java" type="org.eclipse.linuxtools.tmf.ui.editors.UnsortedPropertySheetPage">
+        <filter comment="The Eclipse platform still mistakingly marks PropertySheetPage as @noextend. See bug 109617. " id="571473929">
+            <message_arguments>
+                <message_argument value="PropertySheetPage"/>
+                <message_argument value="UnsortedPropertySheetPage"/>
+            </message_arguments>
+        </filter>
+    </resource>
+</component>
index fd5021692de0926ab58e1d2020303093f9179402..fbe1ea0358837bb4eea7ecacad031b1fdd0d0a43 100644 (file)
@@ -313,7 +313,7 @@ public class CustomTxtTrace extends TmfTrace implements ITmfEventParser {
                             }
                         }
                     }
-                    if (! processed) {
+                    if (!processed && currentInput != null) {
                         final Matcher matcher = currentInput.getPattern().matcher(line);
                         if (matcher.find()) {
                             event.processGroups(currentInput, matcher);
index c275eee8318021459e654a0c30d90d91d45d19d4..9109e6dfd58da2e172152f9f836067fd33f5af52 100644 (file)
@@ -496,8 +496,8 @@ public class CustomTxtParserInputWizardPage extends WizardPage {
                 selectedLine.dispose();
             }
             if (!(event.getSelection().isEmpty()) && event.getSelection() instanceof IStructuredSelection) {
-                IStructuredSelection selection = (IStructuredSelection) event.getSelection();
-                InputLine inputLine = (InputLine) selection.getFirstElement();
+                IStructuredSelection sel = (IStructuredSelection) event.getSelection();
+                InputLine inputLine = (InputLine) sel.getFirstElement();
                 selectedLine = new Line(lineContainer, getName(inputLine), inputLine);
                 lineContainer.layout();
                 lineScrolledComposite.setMinSize(lineContainer.computeSize(SWT.DEFAULT, SWT.DEFAULT).x, lineContainer.computeSize(SWT.DEFAULT, SWT.DEFAULT).y - 1);
@@ -593,9 +593,9 @@ public class CustomTxtParserInputWizardPage extends WizardPage {
 
     private String getSelectionText() {
         if (this.selection instanceof IStructuredSelection) {
-            Object selection = ((IStructuredSelection)this.selection).getFirstElement();
-            if (selection instanceof IFile) {
-                IFile file = (IFile)selection;
+            Object sel = ((IStructuredSelection) this.selection).getFirstElement();
+            if (sel instanceof IFile) {
+                IFile file = (IFile)sel;
                 BufferedReader reader = null;
                 try {
                     reader = new BufferedReader(new InputStreamReader(file.getContents()));
@@ -756,7 +756,7 @@ public class CustomTxtParserInputWizardPage extends WizardPage {
                                     }
                                 }
                             }
-                            if (! processed) {
+                            if (!processed && currentInput != null) {
                                 matcher = currentInput.getPattern().matcher(log);
                                 if (matcher.find()) {
                                     inputText.setStyleRange(new StyleRange(rawPos, length,
@@ -1287,17 +1287,17 @@ public class CustomTxtParserInputWizardPage extends WizardPage {
             }
             inputLine.columns = new ArrayList<InputData>(inputs.size());
             for (int i = 0; i < inputs.size(); i++) {
-                InputGroup group = inputs.get(i);
+                InputGroup grp = inputs.get(i);
                 InputData inputData = new InputData();
-                if (group.tagCombo.getText().equals(CustomTraceDefinition.TAG_OTHER)) {
-                    inputData.name = group.tagText.getText().trim();
+                if (grp.tagCombo.getText().equals(CustomTraceDefinition.TAG_OTHER)) {
+                    inputData.name = grp.tagText.getText().trim();
                 } else {
-                    inputData.name = group.tagCombo.getText();
-                    if (group.tagCombo.getText().equals(CustomTraceDefinition.TAG_TIMESTAMP)) {
-                        inputData.format = group.tagText.getText().trim();
+                    inputData.name = grp.tagCombo.getText();
+                    if (grp.tagCombo.getText().equals(CustomTraceDefinition.TAG_TIMESTAMP)) {
+                        inputData.format = grp.tagText.getText().trim();
                     }
                 }
-                inputData.action = group.actionCombo.getSelectionIndex();
+                inputData.action = grp.actionCombo.getSelectionIndex();
                 inputLine.columns.add(inputData);
             }
         }
index f9b443c99f97188a87302d22e91b46b5abd0d9ed..66560894deb62ca4803792cb1faed5b8816834b4 100644 (file)
@@ -123,8 +123,8 @@ public class CustomTxtParserOutputWizardPage extends WizardPage {
         super.dispose();
     }
 
-    private void loadDefinition(final CustomTxtTraceDefinition definition) {
-        for (final OutputColumn outputColumn : definition.outputs) {
+    private void loadDefinition(final CustomTxtTraceDefinition def) {
+        for (final OutputColumn outputColumn : def.outputs) {
             final Output output = new Output(outputsContainer, outputColumn.name);
             outputs.add(output);
         }
index 5ecaa03afd145911f436af28727a23c220ebdb46..be8e104f41ad9ddaa15855e2b17be81f650d50f5 100644 (file)
@@ -68,8 +68,8 @@ public class CustomTxtParserWizard extends Wizard implements INewWizard {
     }
 
     @Override
-    public void init(IWorkbench workbench, IStructuredSelection selection) {
-        this.selection = selection;
+    public void init(IWorkbench workbench, IStructuredSelection sel) {
+        this.selection = sel;
     }
 
 }
index b01d55403aa09b108833627026ab0b188bbfc5ce..d6d0398470691a6df3faddfaac9cfc56a1f07f72 100644 (file)
@@ -551,8 +551,8 @@ public class CustomXmlParserInputWizardPage extends WizardPage {
                 selectedElement.dispose();
             }
             if (!(event.getSelection().isEmpty()) && event.getSelection() instanceof IStructuredSelection) {
-                IStructuredSelection selection = (IStructuredSelection) event.getSelection();
-                InputElement inputElement = (InputElement) selection.getFirstElement();
+                IStructuredSelection sel = (IStructuredSelection) event.getSelection();
+                InputElement inputElement = (InputElement) sel.getFirstElement();
                 selectedElement = new ElementNode(elementContainer, inputElement);
                 elementContainer.layout();
                 elementScrolledComposite.setMinSize(elementContainer.computeSize(SWT.DEFAULT, SWT.DEFAULT).x,
@@ -692,9 +692,9 @@ public class CustomXmlParserInputWizardPage extends WizardPage {
     private String getSelectionText() {
         InputStream inputStream = null;
         if (this.selection instanceof IStructuredSelection) {
-            Object selection = ((IStructuredSelection) this.selection).getFirstElement();
-            if (selection instanceof IFile) {
-                IFile file = (IFile) selection;
+            Object sel = ((IStructuredSelection) this.selection).getFirstElement();
+            if (sel instanceof IFile) {
+                IFile file = (IFile) sel;
                 try {
                     inputStream = file.getContents();
                 } catch (CoreException e) {
@@ -773,10 +773,6 @@ public class CustomXmlParserInputWizardPage extends WizardPage {
         }
     }
 
-    private void updatePreviews() {
-        updatePreviews(false);
-    }
-
     private void initValues() {
         timeStampValue = null;
         timeStampFormat = null;
@@ -784,7 +780,7 @@ public class CustomXmlParserInputWizardPage extends WizardPage {
         logEntryFound = false;
     }
 
-    private void updatePreviews(boolean updateAll) {
+    private void updatePreviews() {
         if (inputText == null) {
             // early update during construction
             return;
@@ -928,10 +924,10 @@ public class CustomXmlParserInputWizardPage extends WizardPage {
 
                     @Override
                     public void widgetSelected(SelectionEvent e) {
-                        InputElement parent = ElementNode.this.inputElement.parentElement;
-                        while (parent != null) {
-                            parent.logEntry = false;
-                            parent = parent.parentElement;
+                        InputElement parentElem = ElementNode.this.inputElement.parentElement;
+                        while (parentElem != null) {
+                            parentElem.logEntry = false;
+                            parentElem = parentElem.parentElement;
                         }
                     }
                 });
index 1c83ba9572c4fb55e9a07a0dead935f3a0103441..5d6bc8739ceb7fe7cf8b5d7544700e0ac66f7646 100644 (file)
@@ -124,8 +124,8 @@ public class CustomXmlParserOutputWizardPage extends WizardPage {
         super.dispose();
     }
 
-    private void loadDefinition(final CustomTraceDefinition definition) {
-        for (final OutputColumn outputColumn : definition.outputs) {
+    private void loadDefinition(final CustomTraceDefinition def) {
+        for (final OutputColumn outputColumn : def.outputs) {
             final Output output = new Output(outputsContainer, outputColumn.name);
             outputs.add(output);
         }
index fc17585b8c27cf8bbc2885b8184a087db139a22c..ebf35cf4418550b45656193930a3ee221f549cd4 100644 (file)
@@ -72,8 +72,8 @@ public class CustomXmlParserWizard extends Wizard implements INewWizard {
     }
 
     @Override
-    public void init(IWorkbench workbench, IStructuredSelection selection) {
-        this.selection = selection;
+    public void init(IWorkbench workbench, IStructuredSelection sel) {
+        this.selection = sel;
     }
 
 }
index 4fb945d7a81a8c0124cd11eaf2d464a4882fcfcf..e5918136e0f69cd126e17c801123642d02f99790 100644 (file)
@@ -811,9 +811,8 @@ public class ImportTraceWizardPage extends WizardResourceImportPage {
             return false;
         }
 
-        String sourceDirPath;
         try {
-            sourceDirPath = sourceDir.getCanonicalPath();
+            sourceDir.getCanonicalPath();
         } catch (IOException e) {
             MessageDialog.openInformation(getContainer().getShell(), Messages.ImportTraceWizard_Information,
                     Messages.ImportTraceWizard_InvalidTraceDirectory);
@@ -835,7 +834,7 @@ public class ImportTraceWizardPage extends WizardResourceImportPage {
         }
 
         if (fileSystemObjects.size() > 0) {
-            boolean ok = importResources(sourceDirPath, fileSystemObjects);
+            boolean ok = importResources(fileSystemObjects);
             String traceBundle = null;
             String traceTypeId = null;
             String traceIcon = null;
@@ -919,7 +918,7 @@ public class ImportTraceWizardPage extends WizardResourceImportPage {
         return false;
     }
 
-    private boolean importResources(String rootDirectory, Map<String, File> fileSystemObjects) {
+    private boolean importResources(Map<String, File> fileSystemObjects) {
 
         // Determine the sorted canonical list of items to import
         List<File> fileList = new ArrayList<File>();
index 60e90767fa6d34cca1d4b2d07403733630457bc8..4ccc41f2c1c9d532a4c1687c48e83c561c5d8b7e 100644 (file)
@@ -236,15 +236,15 @@ public class TmfEventsCache {
         final int index = current * fCache.length;
 
         class DataRequest extends TmfDataRequest {
-            ITmfFilter fFilter;
-            int fRank;
-            int fIndex;
+            ITmfFilter requestFilter;
+            int requestRank;
+            int requestIndex;
 
-            DataRequest(Class<? extends ITmfEvent> dataType, ITmfFilter filter, int start, int nbRequested) {
+            DataRequest(Class<? extends ITmfEvent> dataType, ITmfFilter reqFilter, int start, int nbRequested) {
                 super(dataType, start, nbRequested);
-                fFilter = filter;
-                fRank = start;
-                fIndex = index;
+                requestFilter = reqFilter;
+                requestRank = start;
+                requestIndex = index;
             }
 
             @Override
@@ -253,18 +253,18 @@ public class TmfEventsCache {
                 if (isCancelled()) {
                     return;
                 }
-                if (fRank >= rank) {
+                if (requestRank >= rank) {
                     cancel();
                     return;
                 }
-                fRank++;
-                if (fFilter.matches(event)) {
-                    fIndex++;
+                requestRank++;
+                if (requestFilter.matches(event)) {
+                    requestIndex++;
                 }
             }
 
             public int getFilteredIndex() {
-                return fIndex;
+                return requestIndex;
             }
         }
 
index 00a85c1fdd2f0bb298131926e63a849583490b5e..fdd2add807492ea5f5d761575a666e053e3f45a5 100644 (file)
@@ -1030,8 +1030,8 @@ public class TmfEventsTable extends TmfComponent implements IGotoMarker, IColorS
                 if (fHeaderState == HeaderState.SEARCH) {
                     stopSearchThread();
                     final TmfFilterAndNode filter = new TmfFilterAndNode(null);
-                    for (final TableColumn column : fTable.getColumns()) {
-                        final Object filterObj = column.getData(Key.SEARCH_OBJ);
+                    for (final TableColumn col : fTable.getColumns()) {
+                        final Object filterObj = col.getData(Key.SEARCH_OBJ);
                         if (filterObj instanceof ITmfFilterTreeNode) {
                             filter.addChild((ITmfFilterTreeNode) filterObj);
                         }
@@ -1048,8 +1048,8 @@ public class TmfEventsTable extends TmfComponent implements IGotoMarker, IColorS
                     }
                 } else if (fHeaderState == HeaderState.FILTER) {
                     final TmfFilterAndNode filter = new TmfFilterAndNode(null);
-                    for (final TableColumn column : fTable.getColumns()) {
-                        final Object filterObj = column.getData(Key.FILTER_OBJ);
+                    for (final TableColumn col : fTable.getColumns()) {
+                        final Object filterObj = col.getData(Key.FILTER_OBJ);
                         if (filterObj instanceof ITmfFilterTreeNode) {
                             filter.addChild((ITmfFilterTreeNode) filterObj);
                         }
index ddb1704fad4795454be9fab8a354e75e98fefd69..5d681099beec72e91fe78d78629551fb8ab7a0c8 100644 (file)
@@ -320,16 +320,16 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
     private class ItemizeThread extends Thread {
 
         private final TimeChartAnalysisEntry fTimeAnalysisEntry;
-        private final long fStartTime;
-        private final long fStopTime;
+        private final long startTime;
+        private final long stopTime;
         private final long fMaxDuration;
 
         private ItemizeThread(TimeChartAnalysisEntry timeAnalysisEntry, long startTime, long stopTime) {
             super("Itemize Thread:" + timeAnalysisEntry.getName()); //$NON-NLS-1$
             fTimeAnalysisEntry = timeAnalysisEntry;
-            fStartTime = startTime;
-            fStopTime = stopTime;
-            fMaxDuration = 3 * (fStopTime - fStartTime) / fDisplayWidth;
+            this.startTime = startTime;
+            this.stopTime = stopTime;
+            fMaxDuration = 3 * (stopTime - startTime) / fDisplayWidth;
         }
 
         @Override
@@ -343,15 +343,15 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
             boolean hasNext = true;
             while (hasNext) {
                 synchronized (timeAnalysisEntry) {
-                    while (hasNext = iterator.hasNext()) {
+                    while ((hasNext = iterator.hasNext()) == true) {
                         event = (TimeChartEvent) iterator.next();
-                        if (event.getTime() + event.getDuration() > fStartTime && event.getTime() < fStopTime && event.getDuration() > fMaxDuration
+                        if (event.getTime() + event.getDuration() > startTime && event.getTime() < stopTime && event.getDuration() > fMaxDuration
                                 && event.getNbEvents() > 1) {
                             break;
                         }
                     }
                 }
-                if (hasNext) {
+                if (hasNext && event != null) {
                     if (event.getItemizedEntry() == null) {
                         itemizeEvent(event);
                     } else {
@@ -432,12 +432,12 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
             boolean hasNext = true;
             while (!interrupted && hasNext) {
                 synchronized (timeAnalysisEntry) {
-                    while (hasNext = iterator.hasNext()) {
+                    while ((hasNext = iterator.hasNext()) == true) {
                         event = (TimeChartEvent) iterator.next();
                         break;
                     }
                 }
-                if (hasNext) {
+                if (hasNext && event != null) {
                     // TODO possible concurrency problem here with ItemizeJob
                     event.setColorSettingPriority(ColorSettingsManager.PRIORITY_NONE);
                     if (event.getItemizedEntry() != null) {
@@ -459,12 +459,12 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
             boolean hasNext = true;
             while (!interrupted && hasNext) {
                 synchronized (timeAnalysisEntry) {
-                    while (hasNext = iterator.hasNext()) {
+                    while ((hasNext = iterator.hasNext()) == true) {
                         event = (TimeChartEvent) iterator.next();
                         break;
                     }
                 }
-                if (hasNext) {
+                if (hasNext && event != null) {
                     // TODO possible concurrency problem here with ItemizeJob
                     if (event.getItemizedEntry() == null) {
                         decorateEvent(event);
index 944b9b294babb36cb2754f356c37d8a569cdb9ad..0f4f4d6bdc07a3251231a9f8ce11efa80cb1ec5a 100644 (file)
@@ -918,12 +918,6 @@ public class TimeGraphViewer implements ITimeDataProvider, SelectionListener {
         _time1_extSynch = _time1;
     }
 
-    @Override
-    @Deprecated
-    public boolean isCalendarFormat() {
-        return timeFormat == TimeFormat.CALENDAR;
-    }
-
     /**
      * @since 2.0
      */
index 4b75ae8abbeedc39f658778597cb925d92c37dbd..1d8db4c729d7292ecc006fc8823923bc4e3038f8 100644 (file)
@@ -136,13 +136,6 @@ public interface ITimeDataProvider {
      */
     int getTimeSpace();
 
-    /**
-     * @return If the time format is calendar (true) or relative (false)
-     * @deprecated replaced by {@link #getTimeFormat()}
-     */
-    @Deprecated
-    boolean isCalendarFormat();
-
     /**
      * @return the time format, one of:
      * <ul>
index 5e0a117a5aeed2a61dbbe49c94e32cc6570f9c70..227f82c6b508030154fc9f5fa0c081d9f10164ca 100644 (file)
@@ -1762,9 +1762,7 @@ public class TimeGraphControl extends TimeGraphBaseControl implements FocusListe
                 redraw();
                 _dragState = DRAG_NONE;
             } else if (e.button == 3 && DRAG_ZOOM == _dragState) {
-                Point size = getCtrlSize();
                 int nameWidth = _timeProvider.getNameSpace();
-                int x = e.x - nameWidth;
                 if (Math.max(_dragX, _dragX0) > nameWidth && _dragX != _dragX0) {
                     long time0 = getTimeAtX(_dragX0);
                     long time1 = getTimeAtX(_dragX);
index d4b575f0b5b9c6a59d50b260216ec7eeba0ebc13..3cb48fa72ddadfd1cad2b5b2bd19b83d1a4d630e 100644 (file)
@@ -112,7 +112,7 @@ public class TimeGraphScale extends TimeGraphBaseControl implements
     private void calcTimeDelta(int width, double pixelsPerNanoSec) {
         double minDelta = (pixelsPerNanoSec == 0) ? YEAR_IN_NS : width / pixelsPerNanoSec;
         long unit = 1;
-        if (_timeProvider != null && _timeProvider.isCalendarFormat()) {
+        if (_timeProvider != null && _timeProvider.getTimeFormat().equals(TimeFormat.CALENDAR)) {
             if (minDelta > 6 * MONTH_IN_NS) {
                 unit = YEAR_IN_NS;
             } else if (minDelta > 3 * MONTH_IN_NS) {
@@ -299,7 +299,7 @@ public class TimeGraphScale extends TimeGraphBaseControl implements
         _rect0.width = labelWidth;
 
         long time;
-        if (_timeProvider != null && _timeProvider.isCalendarFormat()) {
+        if (_timeProvider != null && _timeProvider.getTimeFormat().equals(TimeFormat.CALENDAR)) {
             time = floorToCalendar(time0, _timeDelta);
         } else {
             time = (time0 / _timeDelta) * _timeDelta;
@@ -310,7 +310,7 @@ public class TimeGraphScale extends TimeGraphBaseControl implements
 
         int y = _rect0.y + _rect0.height;
 
-        if (_timeProvider != null && _timeProvider.isCalendarFormat()) {
+        if (_timeProvider != null && _timeProvider.getTimeFormat().equals(TimeFormat.CALENDAR)) {
             timeDraw.drawAbsHeader(gc, time, absHeaderRect);
         }
 
@@ -329,7 +329,7 @@ public class TimeGraphScale extends TimeGraphBaseControl implements
             if (pixelsPerNanoSec == 0 || time > Long.MAX_VALUE - _timeDelta || _timeDelta == 0) {
                 break;
             }
-            if (_timeProvider != null && _timeProvider.isCalendarFormat()) {
+            if (_timeProvider != null && _timeProvider.getTimeFormat().equals(TimeFormat.CALENDAR)) {
                 if (_timeDelta >= YEAR_IN_NS) {
                     long millis = time / 1000000L;
                     GREGORIAN_CALENDAR.setTime(new Date(millis));
@@ -396,7 +396,7 @@ public class TimeGraphScale extends TimeGraphBaseControl implements
         int numDigits = 5;
         long timeRange = time1 - time0;
 
-        if (_timeProvider.isCalendarFormat()) {
+        if (_timeProvider.getTimeFormat().equals(TimeFormat.CALENDAR)) {
             // Calculate the number of digits to represent the minutes provided
             // 11:222
             // HH:mm:ss
This page took 0.037566 seconds and 5 git commands to generate.