Linked by Thom Holwerda on Tue 2nd Apr 2013 12:23 UTC
ReactOS "The ReactOS project is rapidly developing. One of the developers participating in this project suggested that we re-analyzed the source code, as the code base is growing fast. We were glad to do that. We like this project, and we'll be happy if this article helps the developers to eliminate some bugs. Analysis was performed with the PVS-Studio 5.02 code analyzer."
Thread beginning with comment 557363
To view parent comment, click here.
To read all comments associated with this story, please click here.
RE[4]: Advert?
by moondevil on Tue 2nd Apr 2013 20:37 UTC in reply to "RE[3]: Advert?"
Member since:

I usually advocate to always enable warnings as errors and use a syntactic analyzer in the continuous build server, specially in languages where it is so easy to write insecure code.

Locally a developer can do whatever s/he wants, but into source control, only sane content gets in.

Reply Parent Score: 2

RE[5]: Advert?
by bert64 on Wed 3rd Apr 2013 18:14 in reply to "RE[4]: Advert?"
bert64 Member since:

A number of open source packages ship with warnings-as-errors enabled by default in the build process... This can be EXTREMELY annoying, especially when it isn't easy to turn off...
Warnings can occur due to newer compilers, architecture/os differences etc, and in many cases warnings don't stop the program working. Sure in an ideal world code would all compile without any warnings on any platform, but in reality many warnings are false positives and noone writes perfect code.

Reply Parent Score: 2

RE[6]: Advert?
by ebasconp on Wed 3rd Apr 2013 20:00 in reply to "RE[5]: Advert?"
ebasconp Member since:

Maybe you are right when talking about compiling third-party code you did not write but you are sure enough that it works, but for writing your own code, creating it with 0 warnings means two possitive things:

- You tried to keep your code quality as high as you could.
- You removed some unused variables and verified (and fix) some issues reported by the compiler that be potential errors right now or in the future.

Reply Parent Score: 2