Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Incorrect loop variable #10414

Merged
merged 1 commit into from Mar 23, 2017
Merged

Incorrect loop variable #10414

merged 1 commit into from Mar 23, 2017

Conversation

DieselMachine84
Copy link

I used PVS-Studio static analyzer to check this project. I would like to suggest a variant of the way to fix the error, detected with the help of V534 diagnostic. Description of the diagnostic: https://www.viva64.com/en/examples/v534/

Incorrect variable 'i' from the outer loop is used in condition of the inner loop.

… to suggest a variant of the way to fix the error, detected with the help of V534 diagnostic. Description of the diagnostic: https://www.viva64.com/en/examples/v534/

Incorrect variable 'i' from the outer loop is used in condition of the inner loop.
Copy link
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@janvorli janvorli merged commit 5bcd081 into dotnet:master Mar 23, 2017
jorive pushed a commit to guhuro/coreclr that referenced this pull request May 4, 2017
… to suggest a variant of the way to fix the error, detected with the help of V534 diagnostic. Description of the diagnostic: https://www.viva64.com/en/examples/v534/ (dotnet#10414)

Incorrect variable 'i' from the outer loop is used in condition of the inner loop.
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants