Hello Kevin,

> A quick question to any more-knowledgeable-than-me person on whether there's
> a reason these spots in the calc code base are hard-coded to sal_Int16,

It is sal_uInt16 which is not the same as SCCOL.

> rather than using SCCOL?
>

There are 3 different cases here:

- pivot.hxx may use SCCOL, but I think that the used short can only
have values between 0 and 7 there ( Kohei should know it much better )

- source/filter/excel should always use the type already used there
otherwise the binary import might make problems

- pfiltdlg.cxx uses sal_uInt16 because it is an unsigned datatype and
SCCOL is signed, we could use SCCOL there too


Markus
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to