checking engine with static analyzer

[SmallvilleMC]

Hi, the source code has been tested static analyzer, make somebody bug-report.

Spring RTS Engine Checkup
http://www.viva64.com/en/b/0293/

[gajop]

Seems like you are trying to advertise your static analyzer tool, BUT since you also pointed out to some potential errors (and not just random garbage), and you actually took the time to write an analysis yourself, I will allow this post. Thanks. (pity no one here is likely to buy the licence to re-run the static analysis if this was to be fixed)

[Anarchid]

I've read this on other source earlier today, looks like they've gotten quite a PR campaign going :)

That said, all bugs listed in that post should totally be transcribed to appropriate github issues or pull requests.

I wonder if the gross magnitude of assimp errors is something that's fixed in upstream assimp? Has the assimp upgrade branch gone merged?

[jamerlan]

(and russian version)
http://habrahabr.ru/company/pvs-studio/blog/244865/

[yuritch]

Those guys check opensource projects quite often (and post results to advertise their product of course). Their blog (linked to in jamerlan's comment) contains a lot of such check results. I've read it before and I'd say it's a very useful source of information, despite advertisement purpose.

[Forboding Angel]

I 100% approve of this advertisement method. It's like hey, we have something really cool here, but instead of hiding what it can do for you, here is a full analysis from the tool and things that can help your project too!

That's pretty cool imo.

[Anarchid]

I've investigated the repo a little and it seems abma's assimp update branch is not merged.

[abma]

assimp update wasn't merged because of missing feedback for http://springrts.com/phpbb/viewtopic.ph ... 04#p562504

(regardingless of that i plan to update after a successful 99.0 release)