[Libreoffice] HTML_ColorEntry sal_uLong->sal_uInt32

Chr. Rossmanith ChrRossmanith at gmx.de
Tue May 17 12:14:09 PDT 2011


Hi,
> 	It looks nice :-) then again - I noticed this mess:
>
> #if defined( WNT )
>   __cdecl
> #endif
> #if defined( ICC )
>   _Optlink
> #endif
>
> 	Which (you would hope) would be implied by the extern "C" { } construct
> on Windows - with a SAL_CALL macro annotation instead :-) That won't
> work for ICC - but - our ICC support is almost certainly broken anyway
> and I doubt anyone cares.
>
Find patch attached. Here is what I did:

- changed a variables holding boolean values from int to bool
- used include-what-you-use and created new list of #includes
- introduced SAL_CALL

A question to SAL_CALL: The following command

find . -name "*.cxx"|xargs pcregrep -M 'WNT.*\n.*cdecl'

shows SAL_CALL candidates. But where in LO I can assume that it's fine 
to use SAL_CALL. Everywhere? E.g. autodoc/source/exes/adc_uni/main.cxx? 
Or for the first run only replace occurrences similar enough to the one 
above?

Christina


More information about the LibreOffice mailing list