<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br></span>
I don't even think the Coverity issues are a good set of tasks for newcomers.<br>
<br></blockquote><div>I respectfully disagree, I agree Coverity issues should not be picked by newcomers, but there are a number of them that could be assigned to newcomers with a mentors help.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We mostly have close to zero open issues there anyway.<br></blockquote><div>383 outstanding this morning, is of course not a lot considering the code size, but still enough to do something about it.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
One example is when we recently started to get Coverity/FindBugs issues in Java code.  Many of the FindBugs findings need to be taken with a grain of salt, and fixing them naively means being overly optimistic. (For example, a private field of a class not being used in that class does not necessarily mean it should be removed, as it could be used via reflection.)<br></blockquote><div>That is actually exactly one of the reasons why I like to use such issues, it learns a newcomer to not only lot at the problem in itself, but also consider the sideeffects.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Another example is when the implementation of osl_getSystemPathFromFileURL recently happened to start to throw std::length_error,  That means that it is probably often better to let such an exception lead to std::unexpected -> std::abort and a core dump and backtrace that a developer can act upon, than to catch and somehow handle it, and there by obscure the root cause of a problem.</blockquote><div> I see your point, but do not agree in the general case (there are a lot of examples on this where I do agree), however this again is a good example for a newcomer, and should lead to a status change in coverty (Intended).<br><br></div><div>In general I prefer people to not only submit a patch, but also understand the consequences...and sometimes not submitting a patch, but indicating this is an intented behaivour is also a solution. What I do not like is that such issues are kept open and thus potentially multiple people waste time coming to the same conclusion.<br><br></div><div>rgds<br></div><div>jan i.<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</div></div></blockquote></div><br></div></div>