> On May 26, 2015, at 9:26 AM, Emmanuel Lécharny <[email protected]> wrote: > > Shawn, that is impressive ! Fortress-core number of issues has drop by > more than a factor of 2 (from 2543 down to 1161), with not any blocker > and critical issue. Considering that hundreds of the remaining warnings > are in HTML files, this is quit a huge imrpovement for Fortress. > > I'm sure that many warnings are not necessarily important, and some of > them are for code that is not yours (Options and CmdLineParser). > > Congrats ! > > FTR, I conducted the same kind of cleanup for the API, which is way > cleaner now. Not necessarily a funny job, but as Shawn said, this is a > good way to catch big errors at the same time, by teh simple fact you > review the code in the process. > > I guess you are ready for a release soon, right ?
Emmanuel, the sonarqube tool has done the heavy lift, identifies the problems, and suggests solutions. Its reporting tool is easy to use and (as you pointed out) enabled us to catch significant problems in a very short amount of time. All we have to do is fix the ones that matter. :-) Very beneficial tool, highly recommended. Thanks, Shawn [email protected]
