Page 1 of 1
checking engine with static analyzer
Posted: 03 Dec 2014, 09:05
by 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/
Re: checking engine with static analyzer
Posted: 03 Dec 2014, 10:02
by 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)
Re: checking engine with static analyzer
Posted: 03 Dec 2014, 11:56
by 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?
Spring code review by pvs-studio
Posted: 03 Dec 2014, 12:02
by jamerlan
Re: checking engine with static analyzer
Posted: 03 Dec 2014, 17:53
by 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.
Re: checking engine with static analyzer
Posted: 04 Dec 2014, 05:45
by 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.
Re: checking engine with static analyzer
Posted: 04 Dec 2014, 11:00
by Anarchid
I've investigated the repo a little and it seems abma's assimp update branch is not merged.
Re: checking engine with static analyzer
Posted: 06 Dec 2014, 15:58
by 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)