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

Lionel Elie Mamane lionel at mamane.lu
Thu Jan 20 22:59:05 PST 2011


On Mon, Jan 17, 2011 at 04:58:28PM +0000, Caolán McNamara wrote:
> On Mon, 2011-01-17 at 15:18 +0100, Lionel Elie Mamane wrote:

>> (...) 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, (..)
>> [no] 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.

OK, here's the patch that changes the setters to sal_Int32 then.

-- 
Lionel


More information about the LibreOffice mailing list