You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For instance, an issue is raised here and should not be: cPos = curLineStart = 0;
Why? Because this is a not-uncommon practice, and the intent is clear and should not confuse maintainers. Additionally, there's not a lot to be gained by updating it to:
i = 0;
j = 0;
{code}
and it would perhaps be a regression in terms of readability to update it to
{code}
i = 0;
j = i;
For instance, an issue is raised here and should not be:
cPos = curLineStart = 0;
Why? Because this is a not-uncommon practice, and the intent is clear and should not confuse maintainers. Additionally, there's not a lot to be gained by updating it to:
Relates to RSPEC-1121 Assignments should not be made from within sub-expressions
The text was updated successfully, but these errors were encountered: