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.

-- 
[email protected]  <><, Pseudo Engineer, itinerant idiot

_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to