improve name and description of "Anonymous class may be a named 'static' inner class" inspection

This commit is contained in:
Bas Leijdekkers
2014-09-07 16:10:45 +02:00
parent cbda31e0e6
commit a8b9f79728
2 changed files with 6 additions and 8 deletions
@@ -445,8 +445,8 @@ use.processbuilder.class.problem.descriptor=Use of <code>#ref</code> is non-port
use.sun.classes.display.name=Use of sun.* classes
use.sun.classes.problem.descriptor=Use of Sun-supplied class <code>#ref</code> is non-portable #loc
abstract.class.with.only.one.direct.inheritor.display.name=Abstract class which has only one direct inheritor
anonymous.inner.may.be.named.static.inner.class.display.name=Anonymous inner class may be a named static inner class
anonymous.inner.may.be.named.static.inner.class.problem.descriptor=Anonymous inner class <code>#ref</code> may be a named static inner class #loc
anonymous.inner.may.be.named.static.inner.class.display.name=Anonymous class may be a named 'static' inner class
anonymous.inner.may.be.named.static.inner.class.problem.descriptor=Anonymous class <code>#ref</code> may be a named 'static' inner class #loc
array.length.in.loop.condition.display.name=Array.length in loop condition
array.length.in.loop.condition.problem.descriptor=Check of array <code>#ref</code> in loop condition #loc
large.array.allocation.no.outofmemoryerror.display.name=Large array allocation with no OutOfMemoryError check
@@ -1,11 +1,9 @@
<html>
<body>
This inspection is intended for J2ME and other highly resource constrained environments.
Applying the results of this inspection without consideration might have negative effects on code clarity and design.
<p>
Reports any anonymous inner classes which may safely be made into a named
<b>static</b> inner class. An inner class may be static if it doesn't reference
its enclosing class instance or local variables. A static inner class uses slightly less memory.
Reports any anonymous classes which may safely be made into a named <b>static</b> inner class.
An anonymous class may be <b>static</b> if it doesn't explicitly reference its enclosing instance or local variables from its surrounding method.
A <b>static</b> inner class does not keep an implicit reference to its enclosing instance.
This prevents a common cause of memory leaks and uses less memory per instance of the class.
<!-- tooltip end -->
<p>