analysis.lami: merge control flow on ranges equal zero
authorJonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Wed, 1 Jun 2016 19:50:11 +0000 (15:50 -0400)
committerAlexandre Montplaisir <alexmonthy@efficios.com>
Fri, 3 Jun 2016 23:05:29 +0000 (19:05 -0400)
commitc0794ee73799939d9873cbe68a50a86fed8c4b5d
treedffbb239bf402038fcfa8836810939764135cb95
parent8b0730f435890afe5ad6a61a1880d963ce8d7705
analysis.lami: merge control flow on ranges equal zero

Even thought it is the expected behaviour Coverity/Sonar flags the two
"if"s as a potential code duplication and infers an error.

Let's comply to Coverity/Sonar.

Note: Patrick Tassé did mention to merge the two "if"s during initial
review [1] and it was simply forgotten.

Coverity ID: 115874

[1] https://git.eclipse.org/r/#/c/73239/7/analysis/org.eclipse.tracecompass.analysis.lami.ui/src/org/eclipse/tracecompass/internal/provisional/analysis/lami/ui/format/LamiTimeStampFormat.java

Change-Id: I22cf9499eb53fd547ae3ebfc73161f21db5cac47
Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Reviewed-on: https://git.eclipse.org/r/74314
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@efficios.com>
Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Reviewed-by: Hudson CI
analysis/org.eclipse.tracecompass.analysis.lami.ui/src/org/eclipse/tracecompass/internal/provisional/analysis/lami/ui/format/LamiTimeStampFormat.java
This page took 0.025165 seconds and 5 git commands to generate.