From: Joe E. <jen...@fl...> - 2009-05-20 17:06:36
|
Jeff Hobbs wrote: > On 20/05/2009 7:26 AM, Donal K. Fellows wrote: > > That sort of shortening looks good, but you might also consider using %g > > instead of %f so that very large text and canvas widgets can still scroll. > > I tried that, which causes test failures of a different variety (you get > "1"/"0" instead of "1.0"/"0.0"). See also #2112563. The test suite *used* to expect "1" / "0", and when #2112563 was fixed the test suite had to be updated to expect "1.0" / "0.0" instead. This is arguably a bug in the test suite. > Don noted that that could cause issues > with expr as dividing by float 0.0 (non-error Inf) has different > behavior than int 0 (error). My 2c: I don't think it matters *which* error you get if you divide by zero. [$sb set Inf Inf] is going to do the wrong thing anyway. --Joe English |