diff --git a/java/java-impl/src/inspectionDescriptions/ParameterCanBeLocal.html b/java/java-impl/src/inspectionDescriptions/ParameterCanBeLocal.html index 633244cf1441..f39129eb807f 100644 --- a/java/java-impl/src/inspectionDescriptions/ParameterCanBeLocal.html +++ b/java/java-impl/src/inspectionDescriptions/ParameterCanBeLocal.html @@ -6,7 +6,7 @@ parameter can be removed and its usages replaced with local variables. There is no sense to have such parameter as values passed to it are overwritten. Usually problem appears as a result of refactoring. -
E.g.
+For example:
void test(int p) {
p = 1;
diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/ClassMayBeInterface.html b/plugins/InspectionGadgets/src/inspectionDescriptions/ClassMayBeInterface.html
index e36fde4eb549..412cf8a2459a 100644
--- a/plugins/InspectionGadgets/src/inspectionDescriptions/ClassMayBeInterface.html
+++ b/plugins/InspectionGadgets/src/inspectionDescriptions/ClassMayBeInterface.html
@@ -4,8 +4,7 @@ Reports abstract classes which can be converted to interfaces.
Java doesn't support multiple class inheritance while class can implement multiple interfaces,
so it make sense to use interfaces wherever possible instead of classes.
A class may be converted to an interface if it has no superclass (other
-than Object), has no fields declared that are not static, final, and public, and has no
-methods declared that are not public and abstract and no inner classes that are not public.
+than Object), has only public static final fields, public abstract methods, and public inner classes.
Example:
diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/CloneDeclaresCloneNotSupported.html b/plugins/InspectionGadgets/src/inspectionDescriptions/CloneDeclaresCloneNotSupported.html
index 4f1a5b7dadf2..4104d035b33a 100644
--- a/plugins/InspectionGadgets/src/inspectionDescriptions/CloneDeclaresCloneNotSupported.html
+++ b/plugins/InspectionGadgets/src/inspectionDescriptions/CloneDeclaresCloneNotSupported.html
@@ -1,16 +1,14 @@
-Reports clone() methods which are not declared as throwing
-CloneNotSupportedException. If clone() is not declared
-to possibly throw CloneNotSupportedException, then subclasses which need to
-prohibit cloning will not be able to do so in the standard way. This inspection will not report
-clone() methods declared final,
-or clone() methods on final classes.
+Reports clone() methods that don't declare CloneNotSupportedException.
+If CloneNotSupportedException is not declared, then subclasses will not be able to prohibit cloning
+in the standard way. This inspection does not report clone() methods declared final
+and clone() methods on final classes.
Use the checkbox below to indicate if this inspection should only warn on protected methods.
-In Effective Java, Second Edition (but not in the first edition) it is recommended to omit the CloneNotSupportedException
-declaration on public methods, because methods that don't throw checked exceptions are easier to use.
+ You can disable warnings for public methods if you are following the recommendation given in Effective Java, Second Edition> to omit
+ the exception declaration for public methods to make them easier to use.
Example
diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/CommentedOutCode.html b/plugins/InspectionGadgets/src/inspectionDescriptions/CommentedOutCode.html
index c1cb4912a2c2..1f342fe335d7 100644
--- a/plugins/InspectionGadgets/src/inspectionDescriptions/CommentedOutCode.html
+++ b/plugins/InspectionGadgets/src/inspectionDescriptions/CommentedOutCode.html
@@ -1,7 +1,7 @@
Reports comments that contain Java code.
-Usually commented out code gets outdated very fast and becomes misleading.
+Usually code that is commented out gets outdated very fast and becomes misleading.
As most of the projects use some kind of version control system,
it is better to delete commented out code completely and use VCS history instead.
diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/FinalPrivateMethod.html b/plugins/InspectionGadgets/src/inspectionDescriptions/FinalPrivateMethod.html
index b4691ec556f2..0921dde3bc07 100644
--- a/plugins/InspectionGadgets/src/inspectionDescriptions/FinalPrivateMethod.html
+++ b/plugins/InspectionGadgets/src/inspectionDescriptions/FinalPrivateMethod.html
@@ -1,7 +1,7 @@
-Reports methods declared final and private.
-As private methods cannot be meaningfully overridden because of its visibility, declaring them
+Reports methods Reports methods are marked both final and private.
+As private methods cannot be meaningfully overridden because of their visibility, declaring them
final is redundant.
diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/ThrowablePrintedToSystemOut.html b/plugins/InspectionGadgets/src/inspectionDescriptions/ThrowablePrintedToSystemOut.html
index ffb3a30bbdb7..bd53ed6ee9ad 100644
--- a/plugins/InspectionGadgets/src/inspectionDescriptions/ThrowablePrintedToSystemOut.html
+++ b/plugins/InspectionGadgets/src/inspectionDescriptions/ThrowablePrintedToSystemOut.html
@@ -1,12 +1,13 @@
-Reports any calls to System.out.println() with an exception as an argument.
-This is even worse than calling printStackTrace() on an exception, since it will be unclear where the exception originated.
+Reports calls to System.out.println() with an exception as an argument.
+Using print statements for logging exceptions hides the stack trace from you, which is important when you need to investigate what happened.
+It is recommended to use logger instead.
Calls to System.out.print(), System.err.println() and System.err.print() with an exception argument will also be
reported.
It is better to use logger to log exceptions instead.
-E.g. instead of:
+For example, instead of:
try {
foo();