avoiding cppunit zerodiv correction
Lionel Elie Mamane
lionel at mamane.lu
Fri May 30 01:07:18 PDT 2014
On Fri, May 30, 2014 at 10:01:49AM +0200, Matteo Casalin wrote:
>> I wonder if we should not rather do something like:
>> double dRatioPreV;
>> if( nHeight < 0 )
>> {
>> nHeight = 0;
>> dRatioPreV = 1; // or rather zero? or +infinity? Do we want to
>> // "force" a branch of the next if/else?
>> }
>> else
>> dRatioPreV = ((double) nWidth ) / nHeight;
>> Anybody has a good idea, preferably based on an understanding of
>> what this function does?
> I don't have any comment about your proposal, but just would like to
> add that unless we know for sure that nHeight!=0 at that point, in
> both codes the check should be
> if( nHeight <= 0 )
> in order to solve the division by 0 completely.
Ah yes, good point.
--
Lionel
More information about the LibreOffice
mailing list