[Bug 63154] replace tools/solar.h macros with osl versions

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Feb 20 03:29:25 PST 2014


https://bugs.freedesktop.org/show_bug.cgi?id=63154

--- Comment #58 from Stephan Bergmann <sbergman at redhat.com> ---
(In reply to comment #57)
> I understand your point of view, and probably it's the best idea. It's wrong
> to put sal_uLong definition in sal/types.h?

Yes, that's wrong.  sal_uLong is a "temporary" hack for a problem in code
"above" the URE interface, so it should not find its way into the URE stable
interface.

Looks like there is a handful things (SVLIBRARY. sal_uLong) that best stay in
tools/solar.h for now, even if that means that tools/solar.h cannot be removed
completely for now.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20140220/07a429f2/attachment.html>


More information about the LibreOffice mailing list