New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix V3043 #31

Merged
merged 1 commit into from Nov 27, 2017

Conversation

Projects
None yet
2 participants
@pashcovich
Contributor

pashcovich commented Nov 17, 2017

Hello! I'm a member of the Pinguem.ru competition on finding errors in open source projects. A bug, found using PVS-Studio:

  • The code's operational logic does not correspond with its formatting. The statement is indented to the right, but it is always executed. It is possible that curly brackets are missing. NetworkCommsDotNet LzmaEncoder.cs 1430

  • The code's operational logic does not correspond with its formatting. The statement is indented to the right, but it is always executed. It is possible that curly brackets are missing. NetworkCommsDotNet LzmaEncoder.cs 1470

Fix V3043
 - The code's operational logic does not correspond with its formatting. The statement is indented to the right, but it is always executed. It is possible that curly brackets are missing. NetworkCommsDotNet LzmaEncoder.cs 1430

-  The code's operational logic does not correspond with its formatting. The statement is indented to the right, but it is always executed. It is possible that curly brackets are missing. NetworkCommsDotNet LzmaEncoder.cs 1470
@MarcFletcher

Many thanks for the PR.

@MarcFletcher MarcFletcher merged commit f2ba875 into MarcFletcher:master Nov 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment