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