tmf: Remove redundant null checks
authorAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Thu, 21 Feb 2013 22:34:02 +0000 (17:34 -0500)
committerAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Fri, 22 Feb 2013 19:21:01 +0000 (14:21 -0500)
If a final field is instantiated at declaration, it cannot
ever be null, so null checks are redundant in this case.

Change-Id: Iae8a3064bd209d931196ff256fde9db879ea20aa
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/10577
Tested-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
IP-Clean: Patrick Tasse <patrick.tasse@gmail.com>

14 files changed:
org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/event/types/StructDeclaration.java
org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/trace/CTFTraceReader.java
org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/internal/ctf/core/event/EventDeclaration.java
org.eclipse.linuxtools.lttng2.core/src/org/eclipse/linuxtools/internal/lttng2/core/control/model/impl/BaseEventInfo.java
org.eclipse.linuxtools.lttng2.core/src/org/eclipse/linuxtools/internal/lttng2/core/control/model/impl/ChannelInfo.java
org.eclipse.linuxtools.lttng2.core/src/org/eclipse/linuxtools/internal/lttng2/core/control/model/impl/DomainInfo.java
org.eclipse.linuxtools.lttng2.core/src/org/eclipse/linuxtools/internal/lttng2/core/control/model/impl/SessionInfo.java
org.eclipse.linuxtools.lttng2.core/src/org/eclipse/linuxtools/internal/lttng2/core/control/model/impl/UstProviderInfo.java
org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/views/controlflow/ControlFlowEntry.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/model/TmfExperimentElement.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/model/TmfExperimentFolder.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/model/TmfTraceElement.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/project/model/TmfTraceFolder.java
org.eclipse.linuxtools.tmf.ui/src/org/eclipse/linuxtools/tmf/ui/views/timechart/TimeChartView.java

index d4a01717fbada623d2fcf6edb7c5aa116d3b786f..10469300f72daf212244615340b9bf1edd0f48bb 100644 (file)
@@ -18,7 +18,7 @@ import java.util.List;
 
 /**
  * A CTF structure declaration.
- * 
+ *
  * A structure is similar to a C structure, it is a compound data type that
  * contains other datatypes in fields. they are stored in an hashmap and indexed
  * by names which are strings.
@@ -134,8 +134,7 @@ public class StructDeclaration implements IDeclaration {
     public int hashCode() {
         final int prime = 31;
         int result = 1;
-        result = (prime * result)
-                + ((fieldsList == null) ? 0 : fieldsList.hashCode());
+        result = (prime * result) + fieldsList.hashCode();
         result = (prime * result) + (int) (maxAlign ^ (maxAlign >>> 32));
         return result;
     }
@@ -157,11 +156,7 @@ public class StructDeclaration implements IDeclaration {
             return false;
         }
         StructDeclaration other = (StructDeclaration) obj;
-        if (fieldsList == null) {
-            if (other.fieldsList != null) {
-                return false;
-            }
-        } else if (!fieldsList.equals(other.fieldsList)) {
+        if (!fieldsList.equals(other.fieldsList)) {
             return false;
         }
         if (maxAlign != other.maxAlign) {
index c4ced412062602fc1ebd611b78a1201e86f952e5..d9b3ec434d3192bd4836b35fe05c207d53e45750 100644 (file)
@@ -418,9 +418,7 @@ public class CTFTraceReader {
         final int prime = 31;
         int result = 1;
         result = (prime * result) + (int) (startTime ^ (startTime >>> 32));
-        result = (prime * result)
-                + ((streamInputReaders == null) ? 0 : streamInputReaders
-                        .hashCode());
+        result = (prime * result) + streamInputReaders.hashCode();
         result = (prime * result) + ((trace == null) ? 0 : trace.hashCode());
         return result;
     }
@@ -437,11 +435,7 @@ public class CTFTraceReader {
             return false;
         }
         CTFTraceReader other = (CTFTraceReader) obj;
-        if (streamInputReaders == null) {
-            if (other.streamInputReaders != null) {
-                return false;
-            }
-        } else if (!streamInputReaders.equals(other.streamInputReaders)) {
+        if (!streamInputReaders.equals(other.streamInputReaders)) {
             return false;
         }
         if (trace == null) {
index c4c83bc867cf8edd3736b282661cfbb0e63b2999..035c99f588f8cec36c59678cfcea3b3e6c55d939 100644 (file)
@@ -335,7 +335,7 @@ public class EventDeclaration implements IEventDeclaration {
         result = (prime * result) + ((id == null) ? 0 : id.hashCode());
         result = (prime * result) + ((name == null) ? 0 : name.hashCode());
         result = (prime * result) + ((stream == null) ? 0 : stream.hashCode());
-        result = (prime * result) + ((customAttributes == null) ? 0 : customAttributes.hashCode());
+        result = (prime * result) + customAttributes.hashCode();
         return result;
     }
 
index 9ad619b5011f70020f1ac27a3b85c030aceb7dd6..49ffa38fdcde9347421154a418b48ab5d466e07f 100644 (file)
@@ -68,14 +68,12 @@ public class BaseEventInfo extends TraceInfo implements IBaseEventInfo {
     public BaseEventInfo(BaseEventInfo other) {
         super(other);
         fEventType = other.fEventType;
-        if(other.fFields != null) {
-            for (Iterator<IFieldInfo> iterator = other.fFields.iterator(); iterator.hasNext();) {
-                IFieldInfo field = iterator.next();
-                if (field instanceof FieldInfo) {
-                    fFields.add(new FieldInfo((FieldInfo)field));
-                } else {
-                    fFields.add(field);
-                }
+        for (Iterator<IFieldInfo> iterator = other.fFields.iterator(); iterator.hasNext();) {
+            IFieldInfo field = iterator.next();
+            if (field instanceof FieldInfo) {
+                fFields.add(new FieldInfo((FieldInfo)field));
+            } else {
+                fFields.add(field);
             }
         }
         fFilterExpression = other.fFilterExpression;
@@ -238,7 +236,7 @@ public class BaseEventInfo extends TraceInfo implements IBaseEventInfo {
         final int prime = 31;
         int result = super.hashCode();
         result = prime * result + ((fEventType == null) ? 0 : fEventType.hashCode());
-        result = prime * result + ((fFields == null) ? 0 : fFields.hashCode());
+        result = prime * result + fFields.hashCode();
         result = prime * result + ((fFilterExpression == null) ? 0 : fFilterExpression.hashCode());
         result = prime * result + ((fLogLevel == null) ? 0 : fLogLevel.hashCode());
         return result;
@@ -263,11 +261,7 @@ public class BaseEventInfo extends TraceInfo implements IBaseEventInfo {
         if (fEventType != other.fEventType) {
             return false;
         }
-        if (fFields == null) {
-            if (other.fFields != null) {
-                return false;
-            }
-        } else if (!fFields.equals(other.fFields)) {
+        if (!fFields.equals(other.fFields)) {
             return false;
         }
         if (fFilterExpression == null) {
@@ -297,7 +291,7 @@ public class BaseEventInfo extends TraceInfo implements IBaseEventInfo {
             output.append(fEventType);
             output.append(",level=");
             output.append(fLogLevel);
-            if ((fFields != null) && (!fFields.isEmpty())) {
+            if (!fFields.isEmpty()) {
                 output.append(",Fields=");
                 for (Iterator<IFieldInfo> iterator = fFields.iterator(); iterator.hasNext();) {
                     IFieldInfo field = iterator.next();
index 4a3ae9ca43b7022da5317c0d1cc94fbfb6b1deb6..5ccaf04b61bf14712f21cc64d38f23c0ffa9af95 100644 (file)
@@ -282,7 +282,7 @@ public class ChannelInfo extends TraceInfo implements IChannelInfo {
     public int hashCode() {
         final int prime = 31;
         int result = super.hashCode();
-        result = prime * result + ((fEvents == null) ? 0 : fEvents.hashCode());
+        result = prime * result + fEvents.hashCode();
         result = prime * result + fNumberOfSubBuffers;
         result = prime * result + ((fOutputType == null) ? 0 : fOutputType.hashCode());
         result = prime * result + (fOverwriteMode ? 1231 : 1237);
@@ -309,11 +309,7 @@ public class ChannelInfo extends TraceInfo implements IChannelInfo {
             return false;
         }
         ChannelInfo other = (ChannelInfo) obj;
-        if (fEvents == null) {
-            if (other.fEvents != null) {
-                return false;
-            }
-        } else if (!fEvents.equals(other.fEvents)) {
+        if (!fEvents.equals(other.fEvents)) {
             return false;
         }
         if (fNumberOfSubBuffers != other.fNumberOfSubBuffers) {
index 854f7d688f31dadcedc18ff963037e235282ff8f..b4735793f6adaef796b3cb434bb6a0d0b6fb4fbd 100644 (file)
@@ -123,7 +123,7 @@ public class DomainInfo extends TraceInfo implements IDomainInfo {
     public int hashCode() {
         final int prime = 31;
         int result = super.hashCode();
-        result = prime * result + ((fChannels == null) ? 0 : fChannels.hashCode());
+        result = prime * result + fChannels.hashCode();
         result = prime * result + (fIsKernel ? 1231 : 1237);
         return result;
     }
@@ -144,11 +144,7 @@ public class DomainInfo extends TraceInfo implements IDomainInfo {
             return false;
         }
         DomainInfo other = (DomainInfo) obj;
-        if (fChannels == null) {
-            if (other.fChannels != null) {
-                return false;
-            }
-        } else if (!fChannels.equals(other.fChannels)) {
+        if (!fChannels.equals(other.fChannels)) {
             return false;
         }
         if (fIsKernel != other.fIsKernel) {
index 164da048294661352a4b38611ed70c1693b68ed7..7a0ff2e357210931b7fabea86076543bbdba543b 100644 (file)
@@ -192,7 +192,7 @@ public class SessionInfo extends TraceInfo implements ISessionInfo {
     public int hashCode() {
         final int prime = 31;
         int result = super.hashCode();
-        result = prime * result + ((fDomains == null) ? 0 : fDomains.hashCode());
+        result = prime * result + fDomains.hashCode();
         result = prime * result + (fIsStreamedTrace ? 1231 : 1237);
         result = prime * result + ((fSessionPath == null) ? 0 : fSessionPath.hashCode());
         result = prime * result + ((fState == null) ? 0 : fState.hashCode());
@@ -215,11 +215,7 @@ public class SessionInfo extends TraceInfo implements ISessionInfo {
             return false;
         }
         SessionInfo other = (SessionInfo) obj;
-        if (fDomains == null) {
-            if (other.fDomains != null) {
-                return false;
-            }
-        } else if (!fDomains.equals(other.fDomains)) {
+        if (!fDomains.equals(other.fDomains)) {
             return false;
         }
         if (fIsStreamedTrace != other.fIsStreamedTrace) {
index 8ac03fbb9dc314783e540c52bbd75205728c07b7..0169d2d1840f62eb2c8343a22e6e81e29ecd2af2 100644 (file)
@@ -132,7 +132,7 @@ public class UstProviderInfo extends TraceInfo implements IUstProviderInfo {
     public int hashCode() {
         final int prime = 31;
         int result = super.hashCode();
-        result = prime * result + ((fEvents == null) ? 0 : fEvents.hashCode());
+        result = prime * result + fEvents.hashCode();
         result = prime * result + fPid;
         return result;
     }
@@ -153,11 +153,7 @@ public class UstProviderInfo extends TraceInfo implements IUstProviderInfo {
             return false;
         }
         UstProviderInfo other = (UstProviderInfo) obj;
-        if (fEvents == null) {
-            if (other.fEvents != null) {
-                return false;
-            }
-        } else if (!fEvents.equals(other.fEvents)) {
+        if (!fEvents.equals(other.fEvents)) {
             return false;
         }
         if (fPid != other.fPid) {
index 01ac0ab169d5d5581d1b0149533bea2ea9101a84..22cede4c87f72ad828df2ffb6e382ef282e09c72 100644 (file)
@@ -72,7 +72,7 @@ public class ControlFlowEntry implements ITimeGraphEntry {
 
     @Override
     public boolean hasChildren() {
-        return fChildren != null && fChildren.size() > 0;
+        return fChildren.size() > 0;
     }
 
     @Override
index 9d6f1c0ce78f1597fa44754b628a70b3198cb77c..33448a383d3aafe8c8d13002db944cb74feb8691 100644 (file)
@@ -171,7 +171,7 @@ public class TmfExperimentElement extends TmfProjectModelElement implements IPro
      */
     @Override
     public IPropertyDescriptor[] getPropertyDescriptors() {
-        return (sfDescriptors != null) ? Arrays.copyOf(sfDescriptors, sfDescriptors.length) : null;
+        return Arrays.copyOf(sfDescriptors, sfDescriptors.length);
     }
 
     /*
index d78b2196bc7f3df6819c018c3977a4ff2799f0b0..490d8d1bfc5329205e6043c7a134d2537085fb38 100644 (file)
@@ -118,7 +118,7 @@ public class TmfExperimentFolder extends TmfProjectModelElement implements IProp
      */
     @Override
     public IPropertyDescriptor[] getPropertyDescriptors() {
-        return (sfDescriptors != null) ? Arrays.copyOf(sfDescriptors, sfDescriptors.length) : null;
+        return Arrays.copyOf(sfDescriptors, sfDescriptors.length);
     }
 
     /*
index f1cee49ca3d014c174e3df8d73bb724b7ae6246b..2b3ca409d7f5c1f3cf04eeb09554c74ef4362a0a 100644 (file)
@@ -527,7 +527,7 @@ public class TmfTraceElement extends TmfProjectModelElement implements IActionFi
      */
     @Override
     public IPropertyDescriptor[] getPropertyDescriptors() {
-        return (sfDescriptors != null) ? Arrays.copyOf(sfDescriptors, sfDescriptors.length) : null;
+        return Arrays.copyOf(sfDescriptors, sfDescriptors.length);
     }
 
     /*
index 483d2075bf6d2b419d306c2f81c01ffefebcf7fc..6e32652181e880f58c1e0bdb055bcda94800ce45 100644 (file)
@@ -141,7 +141,7 @@ public class TmfTraceFolder extends TmfProjectModelElement implements IPropertyS
      */
     @Override
     public IPropertyDescriptor[] getPropertyDescriptors() {
-        return (sfDescriptors != null) ? Arrays.copyOf(sfDescriptors, sfDescriptors.length) : null;
+        return Arrays.copyOf(sfDescriptors, sfDescriptors.length);
     }
 
     /*
index 5d681099beec72e91fe78d78629551fb8ab7a0c8..5ca3a0a93b70fabf3e20c0253aa26dde233ff3c9 100644 (file)
@@ -612,9 +612,6 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
      */
     @TmfSignalHandler
     public void traceOpened(TmfTraceOpenedSignal signal) {
-        if (fTimeAnalysisEntries == null) {
-            return;
-        }
         final ITmfTrace trace = signal.getTrace();
         final IFile bookmarksFile = signal.getBookmarksFile();
         TimeChartAnalysisEntry timeAnalysisEntry = null;
@@ -643,9 +640,6 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
      */
     @TmfSignalHandler
     public void traceClosed(TmfTraceClosedSignal signal) {
-        if (fTimeAnalysisEntries == null) {
-            return;
-        }
         final ITmfTrace trace = signal.getTrace();
         for (int i = 0; i < fTimeAnalysisEntries.size(); i++) {
             if (fTimeAnalysisEntries.get(i).getTrace().equals(trace)) {
@@ -674,7 +668,7 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
      */
     @TmfSignalHandler
     public void traceSelected(TmfTraceSelectedSignal signal) {
-        if (signal.getSource() != this && fTimeAnalysisEntries != null) {
+        if (signal.getSource() != this) {
             ITmfTrace trace = signal.getTrace();
             for (int i = 0; i < fTimeAnalysisEntries.size(); i++) {
                 if (fTimeAnalysisEntries.get(i).getTrace().equals(trace)) {
@@ -694,9 +688,6 @@ public class TimeChartView extends TmfView implements ITimeGraphRangeListener, I
      */
     @TmfSignalHandler
     public void traceUpdated(TmfTraceUpdatedSignal signal) {
-        if (fTimeAnalysisEntries == null) {
-            return;
-        }
         final ITmfTrace trace = signal.getTrace();
         for (int i = 0; i < fTimeAnalysisEntries.size(); i++) {
             TimeChartAnalysisEntry timeAnalysisEntry = fTimeAnalysisEntries.get(i);
This page took 0.033865 seconds and 5 git commands to generate.