sal / gcc conditionals ...
Michael Meeks
michael.meeks at suse.com
Mon Feb 13 09:45:35 PST 2012
Hi there,
Alexander just removed a load of gcc conditionals from master, but
there were a few I dropped from his patch to get some further input from
you.
We've decided (I think) in
https://bugs.freedesktop.org/show_bug.cgi?id=45131 that 4.0.1
* cppu/inc/cppu/macros.hxx:
CPPU_GCC3_ALIGN - uh ?
I assume this should be just CPPU_GCC_ALIGN and with only one
definition in today's world ?
* sal/inc/rtl/instance.hxx:
#if (__GNUC__ >= 4)
Do we need all those conditionals ? or do we want a
#if defined (__GCC__) there ?
* sal/inc/sal/config.h
We have a conditional for SAL_ISO_CONST, but do we ever even use that
thing ? :-) if not, can we just drop it ? breaks source compat, but ...
if we didn't use it ;-)
Overall, it's quite encouraging to see the improving C++ support of
gcc, and the ripping out of lots of legacy awfulness to work around it.
Thanks :-)
Michael.
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list