rspec/rules/S2189/comments-and-links.adoc

18 lines
790 B
Plaintext

=== on 3 Jan 2017, 17:35:46 Pierre-Yves Nicolas wrote:
Isn't there some overlap with RSPEC-2251, even if RSPEC-2251 is not strictly speaking detecting infinite loops?
=== on 3 Jan 2017, 18:04:08 Ann Campbell wrote:
\[~pierre-yves.nicolas] conceptually, maybe there is, but not semantically. In fact, this got split into two distinct cases/rules because of the semantic difference between
``++while(true)++`` (truly infinite loop) and ``++for (int i=0; i < 10; i--)++`` (inadvertently long loop)
And maybe I'm remembering incorrectly, but I have the sense that the discussion that lead to the split was with you. :)
As a side note, I've updated the code samples to remove the inadvertently long loop.
=== on 5 Jan 2017, 09:10:37 Pierre-Yves Nicolas wrote:
Thanks [~ann.campbell.2]!