<div dir="ltr">Really cool Robert.  I'd love to get TravisCI setup for osgEarth as well, we might pick your brain on it here soon.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 21, 2016 at 5:03 AM Robert Osfield <<a href="mailto:robert.osfield@gmail.com">robert.osfield@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">With help form Ralf Habacker and Jordi Torress we now have Travis<br>
build testing and Coverity scan up and running.  If you got to the<br>
OSG's github page :<br>
<br>
   <a href="https://github.com/openscenegraph/OpenSceneGraph" rel="noreferrer" target="_blank">https://github.com/openscenegraph/OpenSceneGraph</a><br>
<br>
and look down to the README.md you'll see two little bullet icons:<br>
<br>
  [build | passing]  [ coverity | passed 5 new defects]<br>
<br>
You can click on the build icon and it'll take you to the Travis build<br>
logs, and click on the Coverity one and it'll take you to our new<br>
Coverity analysis page.<br>
<br>
For the up coming OpenSceneGraph-3.6 stable release I would like to<br>
improve the Coverity defect count. I have already made dozens of small<br>
fixes to code highlight by Coverity, there is still a long way to go.<br>
For this effort I'd appreciate help from the community :-)<br>
<br>
My personally priority is to squash the High priority ones first, then<br>
move on to Midium and then lower priority ones.  Also defects picked<br>
up in the core libraries like osg, osgUtil, osgDB, osgGA, osgViewer<br>
are ones that will benefit the most users.<br>
<br>
As well as code fixes just reviewing the defect reports and accessing<br>
how crucial it is would be helpful.  The Coverity web interface allows<br>
you to add comments as well as adjust properties/categorization of<br>
defects.  For this you'll need to either log into via your Github<br>
account or create a Coverity account.<br>
<br>
As work on cleaning up defects progresses I will apply the fixes to<br>
master as they come in and then every couple of days merge these<br>
changes as a block to the new coverity_scan branch which will then<br>
fire off a Coverity scan so we can get an update of how the clean up<br>
is progressing.<br>
<br>
When doing fixes please make sure that you fully understand the code<br>
you are modifying and the nature of the defect.  It's all too easy to<br>
fix a warning/defect report on code that was working fine and then<br>
have the new "improved" code introduce a bug that neither produces<br>
warning or a Coverity scan.  I say this as I have made this mistake<br>
myself before, and also spotted such mistakes when merging changes<br>
from others.<br>
<br>
Thanks in advance for you help,<br>
Robert.<br>
_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org" target="_blank">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</blockquote></div>