On Tue, 24 Apr 2018 14:18:50 -0400 "William L. Thomson Jr." <wlt...@obsidian-studios.com> wrote:
> On Tue, 24 Apr 2018 12:09:50 +0200 > Stefan Schmidt <ste...@osg.samsung.com> wrote: > > > https://sonarcloud.io/project/issues?branch=devs%2Fstefan%2Fsonar-test&id=efl&resolved=false > > > > 34441 issues found looks rather noisy to me, but with some filtering > > we might be able to find get the interesting parts out of it. > > Another way to look at false positives or noise. If the code throws > off some static analyzers. How human readable/parseable is the code? > If a static analyzer cannot follow a code path. Maybe humans will > have a hard time just the same. At least something worth considering > IMHO. Yet another way to look at it. Is it a good thing pass something easily? Usually the harder, more it takes to pass something, the better. Surely not always, but in most test situations, I think that holds true. Why I like the Coverage aspect of Sonar. It is easier to pass analyzers than it is to pass code coverage tests. Sonar combines it all, so to pass over all. Not only must your code pass the static test. At least 80% must be executed in some form, via tests or actual run. That can be more difficult than passing static analyzer tests! No way around! For Entrance I had Coverity passing long ago, not sure that is good. It will be a bit before I get Entrance passing under Sonar. It was easier to pass Coverity than to pass Sonar. Which I have yet to do. https://scan.coverity.com/projects/obsidian-studiosinc-entrance Entrance is not passing Sonar, because I have yet to run 80% of the code. Though for some that can be off a bit, another passes at 74%. https://sonarcloud.io/dashboard?id=entrance https://sonarcloud.io/dashboard?id=jem It can take considerable time and effort to pass all Sonar metrics. I consider that a good thing for the most part. Not sure one could argue code is worse that passes Sonar. When it comes to QA do you want more or less? Spend less time, less noise, less false? Or more of all and ensure your doing deep QA. -- William L. Thomson Jr.
pgpl0BXEFvB3J.pgp
Description: OpenPGP digital signature
------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
