https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/VTN-71 vtn <p>We should reduce sonar warnings as possible for Lithium release though they are harmless.</p> <p>Operating System: All<br/> Platform: All</p> VTN-71

[Lithium] Reduce sonar warnings.

Bug Resolved Done Unassigned Shigeru Yasuda Fri, 12 Jun 2015 06:29:34 +0000 Thu, 19 Oct 2017 21:24:56 +0000 Fri, 19 Jun 2015 05:13:51 +0000 unspecified VTN Manager 0 0 <p><a href="https://git.opendaylight.org/gerrit/22438" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22438</a> (master)<br/> <a href="https://git.opendaylight.org/gerrit/22557" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22557</a> (stable/lithium)</p> <ul> <li>"equals(Object obj)" and "hashCode()" should be overridden in pairs</li> <li>Throwable and Error classes should not be caught</li> </ul> <p><a href="https://git.opendaylight.org/gerrit/22441" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22441</a> (master)<br/> <a href="https://git.opendaylight.org/gerrit/22558" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22558</a> (stable/lithium)</p> <ul> <li>Security - Array is stored directly</li> <li>Methods named "equals" should override Object.equals(Object)</li> <li>Throwable.printStackTrace(...) should never be called</li> <li>"object == null" should be used instead of "object.equals(null)"</li> </ul> <p><a href="https://git.opendaylight.org/gerrit/22563" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22563</a> (master)<br/> <a href="https://git.opendaylight.org/gerrit/22664" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22664</a> (stable/lithium)</p> <ul> <li>Unused private fields should be removed</li> <li>Unused local variables should be removed</li> <li>Avoid Throwing Null Pointer Exception</li> <li>Constructor Calls Overridable Method</li> <li>Try-catch blocks should not be nested</li> <li>Collapsible "if" statements should be merged</li> <li>Utility classes should not have a public constructor</li> <li>Unused formal parameter</li> <li>Unnecessary Local Before Return</li> <li>Nested blocks of code should not be left empty</li> <li>Declarations should use Java collection interfaces such as "List"<br/> rather than specific implementation classes such as "LinkedList"</li> <li>Generic exceptions Error, RuntimeException, Throwable and Exception<br/> should never be thrown</li> </ul> <p><a href="https://git.opendaylight.org/gerrit/22961" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22961</a> (master)<br/> <a href="https://git.opendaylight.org/gerrit/22918" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/22918</a> (stable/lithium)</p> <ul> <li>Missing Static Method In Non Instantiatable Class</li> <li>Unnecessary Local Before Return</li> <li>Expressions should not be too complex</li> <li>Methods should not be too complex</li> <li>Declarations should use Java collection interfaces such as "List"<br/> rather than specific implementation classes such as "LinkedList"</li> <li>Exception handlers should preserve the original exception</li> </ul> Development External issue ID 3684 External issue URL ODL SR Target Milestone Rank 0|i02ten: