[C++-sig] Re: Boost.Python in SUSE 9.0 or SUSE 9.1?

[Ralf W. Grosse-Kunstleve]

--- David Abrahams <dave at boost-consulting.com> wrote:
> Chris Hoeppler <hoeppler at diener.iap.physik.uni-tuebingen.de> writes:
> 
> > FWIW, the compiler is happy with  "catch(exception_cref &e)", though I do
> > --------------------------------------------------------^
> > not know whether it would be okay to make that change...
> 
> Sure; you can do that in your local copy.

David is telling me:

> > Hi David, I think he was asking if we can make that change in the main
version
.
> > I think we should unless you have objections. I can run tests on all our
> > platforms to make sure it works everywhere. Please let me know.

> Yes, I have objections, at least if you don't make the patch very
> precise and conditional.  It will break on many compilers.  The
> add_reference invocation above it should have already added a
> reference and many compilers (even most GCCs?) will choke on adding a
> reference to a type that is already a reference.

I.e. what we need is a very specific set of #ifdef's that will apply your patch
only under SUSE 9, and only if the compiler with SUSE's patches is used. Here
is an alias that is sometimes helpful in pin-pointing system specific macros:

alias whatdefg++ 'echo '"'"'int main(void){return 0;}'"'"' >
/var/tmp/tmp$$.cpp; g++ -E -v \!* /var/tmp/tmp$$.cpp; rm -f /var/tmp/tmp$$.cpp'

Please use BOOST_WORKAROUND. There are lots of examples in the boost code, e.g.
boost/python/object/value_holder.hpp.

Ralf



		
__________________________________
Do you Yahoo!?
Yahoo! Mail - Helps protect you from nasty viruses.
http://promotions.yahoo.com/new_mail