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."
Order by: Score:
Advert?
by Brendan on Tue 2nd Apr 2013 13:29 UTC
Brendan
Member since:
2005-11-16

Looks like some company felt like using ReactOS to get free advertising to me...

Also wondering how many of these potential problems would've been caught by simply enabling the compiler's warnings.

- Brendan

Reply Score: 1

RE: Advert?
by jimmmy on Tue 2nd Apr 2013 16:00 UTC in reply to "Advert?"
jimmmy Member since:
2012-01-02

I think it would be interesting to know what percentage of errors caught by the analyzer are not caught by other analyzers.

Oh well. I suppose it doesn't matter anyway because I don't develop for Windows.

Reply Score: 2

RE: Advert?
by drcoldfoot on Tue 2nd Apr 2013 16:07 UTC in reply to "Advert?"
drcoldfoot Member since:
2006-08-25

The advertising also is in ReactOS' favor. So I see it as a win on both fronts.

Reply Score: 7

RE[2]: Advert?
by ebasconp on Tue 2nd Apr 2013 16:53 UTC in reply to "RE: Advert?"
ebasconp Member since:
2006-05-09

Agreed,

Advertising or not, I would like to see more static code analysis for more open source projects.

It is good advertising for the product, good advertising for the open source projects and makes you think: "Hey! I can help right there!" ;)

Reply Score: 3

RE[3]: Advert?
by Valhalla on Tue 2nd Apr 2013 17:09 UTC in reply to "RE[2]: Advert?"
Valhalla Member since:
2006-01-24

Yep, apart from possibly having to wade through 'false positives' there's really no downside to running your codebase through a static analyzer.

I know Haiku has used Coverity and as I recall so had ReactOS before this aswell.

Reply Score: 2

RE[4]: Advert?
by moondevil on Tue 2nd Apr 2013 20:37 UTC in reply to "RE[3]: Advert?"
moondevil Member since:
2005-07-08

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 Score: 2

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

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 Score: 2

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

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 Score: 2

RE: Advert?
by ideasman42 on Wed 3rd Apr 2013 03:06 UTC in reply to "Advert?"
ideasman42 Member since:
2007-07-20

Im sure reactos guys already use compiler warnings, so to answer your Q - Probably none,
Compiler warnings often won't find these kinds of mistakes in most cases (of course depends on your compiler - my experience is with gcc,clang,icc,msvc)

Reply Score: 1

Still at 0.3.14? REALLY?
by tidux on Tue 2nd Apr 2013 19:19 UTC
tidux
Member since:
2011-08-13

Not even a 0.0.01 version bump in over a year?

It's dead, Jim.

Reply Score: 3

RE: Still at 0.3.14? REALLY?
by stabbyjones on Tue 2nd Apr 2013 20:36 UTC in reply to "Still at 0.3.14? REALLY?"
stabbyjones Member since:
2008-04-15

That's like saying e17 will never get released.

Reply Score: 7

RE: Still at 0.3.14? REALLY?
by ozonehole on Wed 3rd Apr 2013 03:29 UTC in reply to "Still at 0.3.14? REALLY?"
ozonehole Member since:
2006-01-07

I've been wondering about this myself. But I really hope it's not dead. Although I'm a Linux user 99.9% of the time, there is that rare instance where I've got to go to Windows to do something for which a Linux app doesn't exist. I recently had to retrieve a video file from a dash-cam that formats micro-SD cards in a proprietary format, and the only way to do it was to run a Windows application. That sucks, and the manufacturer (Abee) should be crucified for that, but it's these situations that make a market for ReactOS. If it was usable, I'd put it on a spare partition for those rare situations when it's needed.

Edited 2013-04-03 03:30 UTC

Reply Score: 2