[Libreoffice] [PATCH] Unix 64 bits property type / default value mismatch

Caolán McNamara caolanm at redhat.com
Mon Jan 17 08:58:28 PST 2011


On Mon, 2011-01-17 at 15:18 +0100, Lionel Elie Mamane wrote:
> Well caught, I missed it. Well, we have to make these three places
> agree. As to whether it is in the direction of "C/C++ long" everywhere
> or in the direction of "uno long == sal_Int32" everywhere, frankly I
> don't have enough overview / depth of knowledge into the codebase to
> have an opinion.

Well, we have to assume from that LoadTableWindowData that the values
may have been cached on disk somewhere, and have been assigned a data
type of sal_Int32 for the purposes of file storage. So I say we use
sal_Int32 everywhere for those four "long"s.

C.



More information about the LibreOffice mailing list