Re: [Libreoffice] HTML_ColorEntry sal_uLong->sal_uInt32

Hi Christina,

On Tue, 2011-05-17 at 09:29 +0200, Chr. Rossmanith wrote:
> Am 16.05.2011 17:05, schrieb Michael Meeks:
> > On Fri, 2011-05-06 at 21:53 +0200, Chr. Rossmanith wrote:
> > > I think the variable nColor from struct HTML_ColorEntry in htmlkywd.cxx
> > > can be changed from sal_uLong to sal_uInt32. Continue to change return
..
> Of course. Find it attached.

        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.

        It seems there are a load of other #if defined WNT - conditionals
around __cdecl in similar circumstances that'd be great to fix if you
have the energy :-)

        Thanks !

                Michael.

-- 
 michael.meeks@novell.com  <><, Pseudo Engineer, itinerant idiot