Re: coverity warnings update, 170, 0.04 density
On Sun, 2014-11-02 at 22:39 +0100, Thorsten Behrens wrote: Caolan McNamara wrote: I reckon practically all the swapped arguments are ok except maybe for 1242508. (caolanm-quikee: in 1b23e46051d8cc7c01fd8b4d0ea51bfec145db8e there is.. - pWriteAcc-SetPixel( i, x, aResultColor ); + pWriteAcc-SetPixel( x, y, aResultColor ); and SetPixel takes args named x, y. so are those arguments definitely not the wrong way around or is it actually a real detected problem?) Aye. But SetPixel arg ordering is just being totally silly. Fix pushed though. Yesterday's run is now an all-time low of 0.04 defect density, with just 163 issues remaining to be triaged. 735814 Explicit null dereferenced in sc/source/ui/namedlg/namedefdlg.cxx:240 looks like a real problem where pNewEntry is dereferenced but an above code path assigns NULL to it if I could get a calc guy to look at it. and 705391 Inferred misuse of enum really bugs me. xmloff/source/transform/MergeElemTContext.cxx:235 where XML_ATACTION_MOVE_FROM_ELEM_RNG2ISO_DATETIME is from a completely different enum than all the other cases. It looks completely bogus, but I don't know if it should be some other condition or if that case belongs to another switch. C. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: coverity warnings update, 170, 0.04 density
On Mon, Nov 3, 2014 at 4:24 PM, Caolán McNamara caol...@redhat.com wrote: On Sun, 2014-11-02 at 22:39 +0100, Thorsten Behrens wrote: Caolan McNamara wrote: I reckon practically all the swapped arguments are ok except maybe for 1242508. (caolanm-quikee: in 1b23e46051d8cc7c01fd8b4d0ea51bfec145db8e there is.. - pWriteAcc-SetPixel( i, x, aResultColor ); + pWriteAcc-SetPixel( x, y, aResultColor ); and SetPixel takes args named x, y. so are those arguments definitely not the wrong way around or is it actually a real detected problem?) Aye. But SetPixel arg ordering is just being totally silly. Fix pushed though. Yesterday's run is now an all-time low of 0.04 defect density, with just 163 issues remaining to be triaged. 735814 Explicit null dereferenced in sc/source/ui/namedlg/namedefdlg.cxx:240 looks like a real problem where pNewEntry is dereferenced but an above code path assigns NULL to it if I could get a calc guy to look at it. That looks like one for me. I'll have a look at it. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: coverity warnings update, 170, 0.04 density
On 03.11.2014 16:24, Caolán McNamara wrote: 705391 Inferred misuse of enum really bugs me. xmloff/source/transform/MergeElemTContext.cxx:235 where XML_ATACTION_MOVE_FROM_ELEM_RNG2ISO_DATETIME is from a completely different enum than all the other cases. It looks completely bogus, but I don't know if it should be some other condition or if that case belongs to another switch. that was a nasty case, fixed in commit 76dce95eed6892d729fbea2b990911edd82ded82 ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice