Hi Markus, On Tuesday, 2011-12-13 00:15:08 +0100, Markus Mohrhard wrote:
the attached patch fixes a crash in our rtl::math lib. For std::numeric_limits<int>::min() -nExp is the same as nExp and therefore we would be in the first case and try to access a non existing element.
Good catch. As Stephan said, please add a comment why the additional check is in so it doesn't get "optimized out" if someone comes across and reads the not so obvious code. Eike -- LibreOffice Calc developer. Number formatter stricken i18n transpositionizer. GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3 9E96 2F1A D073 293C 05FD
Attachment:
pgpgFSM3JR4hy.pgp
Description: PGP signature