<div dir="ltr"><div dir="auto">Beyond the report on an accomplished task, I see the joy and the passion on making something good. I can't explain how much your efforts on taking care of QGIS make my everyday work better.<br><br></div><div>Thank you very much.<br></div><div class="gmail_extra"><br><div class="gmail_quote">Il 26/set/2017 06:40, "Tim Sutton" <<a href="mailto:tim@kartoza.com" target="_blank">tim@kartoza.com</a>> ha scritto:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Nyall<br>
<br>
<br>
Thank you so much for doing this work and for your report back! I hope this is just one of many times that we will be able to fund our community members to make substantial improvements to the quality of QGIS!<br>
<br>
Regards<br>
<br>
Tim<br>
<br>
> On 26 Sep 2017, at 9:31 AM, Nyall Dawson <<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>> wrote:<br>
><br>
> Hi PSC,<br>
><br>
> Just reporting in that with PR #5254 [1], the grant work has now been<br>
> completed and we're sitting as close as possible to 100% unit test<br>
> coverage for the geometry primitive classes (i.e. the classes which<br>
> represent the raw geometry types, such as points/multipoints/circular<br>
> strings/ etc).<br>
><br>
> While the purpose of this work wasn't directly to fix bugs, writing<br>
> the tests has allowed me to identify and fix dozens of issues in these<br>
> geometry classes. The fixes range from pedantic (e.g. crashes when<br>
> requesting parts of the geometry with invalid indexes such as negative<br>
> numbers), through to severe (a geometry corruption caused after<br>
> converting a MultiLineString to GML which would definitely cause QGIS<br>
> server to crash on certain requests).<br>
><br>
> Just to ensure that there's no misconceptions here: having 100% test<br>
> coverage doesn't mean the geometry classes are now bug free - rather<br>
> it means that the current logic is protected so that future features,<br>
> fixes, or optimisations will not cause regressions from the current<br>
> behaviour.<br>
><br>
> I'd like the thank the PSC once again for accepting this funding<br>
> request (and the sponsors who made it possible!). In my opinion this<br>
> was a crucial piece of work required to help make future QGIS releases<br>
> much less likely to suffer serious regressions, so I'm very thankful<br>
> that you allowed it to happen!<br>
><br>
> Regards,<br>
> Nyall<br>
><br>
><br>
> [1] <a href="https://github.com/qgis/QGIS/pull/5254" rel="noreferrer" target="_blank">https://github.com/qgis/QGIS/p<wbr>ull/5254</a><br>
> ______________________________<wbr>_________________<br>
> Qgis-psc mailing list<br>
> <a href="mailto:Qgis-psc@lists.osgeo.org" target="_blank">Qgis-psc@lists.osgeo.org</a><br>
> <a href="https://lists.osgeo.org/mailman/listinfo/qgis-psc" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailma<wbr>n/listinfo/qgis-psc</a><br>
<br>
______________________________<wbr>_________________<br>
QGIS-Developer mailing list<br>
<a href="mailto:QGIS-Developer@lists.osgeo.org" target="_blank">QGIS-Developer@lists.osgeo.org</a><br>
List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailma<wbr>n/listinfo/qgis-developer</a><br>
Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailma<wbr>n/listinfo/qgis-developer</a></blockquote></div></div>
</div>