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.me...@novell.com <><, Pseudo Engineer, itinerant idiot _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice