rspec/rules/S1321/flex/rule.adoc
Fred Tingaud 16f6c0aecf
Inline adoc when include has no additional value (#1940)
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.
2023-05-25 14:18:12 +02:00

37 lines
896 B
Plaintext

== Why is this an issue?
Never use ``++with++`` statements, since they decrease readability. When you do not specify a variable's scope, you do not always know where you are setting properties, so your code can be confusing.
=== Noncompliant code example
[source,flex]
----
with (foo) { // Noncompliant
return x; // is it a property of foo or local variable ?
}
----
=== Compliant solution
[source,flex]
----
return foo.x;
----
ifdef::env-github,rspecator-view[]
'''
== Implementation Specification
(visible only on this page)
include::../message.adoc[]
'''
== Comments And Links
(visible only on this page)
=== on 5 Jan 2015, 18:43:02 Ann Campbell wrote:
Note that this subtask variation of SQALE is an exception to the rule - both JavaScript and Flex plugins had implemented the rule with the same legacy key, but with different SQALE settings.
endif::env-github,rspecator-view[]