16f6c0aecf
Inline adoc files when they are included exactly once. Also fix language tags because this inlining gives us better information on what language the code is written in.
48 lines
1.2 KiB
Plaintext
48 lines
1.2 KiB
Plaintext
== Why is this an issue?
|
|
|
|
Doing an operation on a string without using the result of the operation is useless and is certainly due to a misunderstanding.
|
|
|
|
|
|
=== Noncompliant code example
|
|
|
|
[source,javascript]
|
|
----
|
|
var str = "..."
|
|
str.toUpperCase(); // Noncompliant
|
|
----
|
|
|
|
|
|
=== Compliant solution
|
|
|
|
[source,javascript]
|
|
----
|
|
var str = "..."
|
|
str = str.toUpperCase();
|
|
----
|
|
|
|
ifdef::env-github,rspecator-view[]
|
|
|
|
'''
|
|
== Implementation Specification
|
|
(visible only on this page)
|
|
|
|
=== Message
|
|
|
|
${objectName} is an immutable object; you must either store or return the result of the operation.
|
|
|
|
|
|
'''
|
|
== Comments And Links
|
|
(visible only on this page)
|
|
|
|
=== on 7 Aug 2013, 16:52:44 Freddy Mallet wrote:
|
|
Is implemented by \http://jira.codehaus.org/browse/SONARJAVA-242
|
|
|
|
=== on 27 Feb 2015, 09:27:52 Freddy Mallet wrote:
|
|
FYI [~ann.campbell.2], KlockWorks has associated this rule to "CWE-391: Unchecked Error Condition". I'm not convinced by such association but just wanted to notify you: \http://docs.klocwork.com/Insight-10.0/Checkers:RI.IGNOREDCALL
|
|
|
|
=== on 27 Feb 2015, 16:34:02 Ann Campbell wrote:
|
|
\[~freddy.mallet] I'm also (extremely!) unconvinced, but thanks for the notification.
|
|
|
|
endif::env-github,rspecator-view[]
|