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
Hi Caolán,
On Thursday, 2014-10-30 16:03:53 +, Caolán McNamara wrote:
> Nearly all the other "swapped arguments" are around column/rows possibly
> being swapped, especially in calc, e.g. a lot of them are like 1242513
> bChartRowAsHdr passed to bCol on line 3176
> ScCellRangesBase::setRowDesc
So, we now have < 200 non-(intentional/false positive) warnings
remaining in coverity from an original count of approx 11 thousand or
thereabouts.
the biggest remaining categories are...
50 "Tainted scalars"
23 "Uncaught exceptions"
19 "Unchecked returns"
19 "divide by zeros"
14 "Swapped argument
