[type_traits] is_base_and_derived on Borland 5.8.1

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[type_traits] is_base_and_derived on Borland 5.8.1

Daniel James
Hi,

is_base_and_derived isn't working on Borland 5.8.1, as you can see if
you compare the test results for 5.8.1 (http://tinyurl.com/nru5r) and
5.6.4 (http://tinyurl.com/mh4s9). The attached patch might fix it.
Please, could someone with access to the compiler try it out?

thanks,

Daniel

Index: is_base_and_derived.hpp
===================================================================
RCS file: /cvsroot/boost/boost/boost/type_traits/is_base_and_derived.hpp,v
retrieving revision 1.16
diff -u -r1.16 is_base_and_derived.hpp
--- is_base_and_derived.hpp 7 Oct 2005 14:26:38 -0000 1.16
+++ is_base_and_derived.hpp 2 Mar 2006 18:00:18 -0000
@@ -23,7 +23,7 @@
 
 namespace detail {
 
-#if !BOOST_WORKAROUND(__BORLANDC__, <= 0x570) \
+#if !BOOST_WORKAROUND(__BORLANDC__, BOOST_TESTED_AT(0x570)) \
  && !BOOST_WORKAROUND(__SUNPRO_CC , BOOST_TESTED_AT(0x540)) \
  && !BOOST_WORKAROUND(__EDG_VERSION__, <= 243) \
  && !BOOST_WORKAROUND(__DMC__, BOOST_TESTED_AT(0x840))

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [type_traits] is_base_and_derived on Borland 5.8.1

Nicola Musatti
Daniel James wrote:
> Hi,
>
> is_base_and_derived isn't working on Borland 5.8.1, as you can see if
> you compare the test results for 5.8.1 (http://tinyurl.com/nru5r) and
> 5.6.4 (http://tinyurl.com/mh4s9). The attached patch might fix it.
> Please, could someone with access to the compiler try it out?

I think you should check out the work being done by Alisdair Meredith to
  enable BCB 2006 support in the Boost CVS repository in time for the
fortcoming 1.34 release. It's all described in recent threads in this
newsgroup. If you're interested in Boost 1.33.1 the bcbboost project
(bcbboost.sourceforge.net) is trying to do the same for the current release.

Cheers,
Nicola Musatti

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [type_traits] is_base_and_derived on Borland 5.8.1

AlisdairM
Nicola Musatti wrote:

> Daniel James wrote:
> > Hi,
> >
> > is_base_and_derived isn't working on Borland 5.8.1, as you can see
> > if you compare the test results for 5.8.1
> > (http://tinyurl.com/nru5r) and 5.6.4 (http://tinyurl.com/mh4s9).
> > The attached patch might fix it.  Please, could someone with access
> > to the compiler try it out?
>
> I think you should check out the work being done by Alisdair Meredith
> to   enable BCB 2006 support in the Boost CVS repository in time for
> the fortcoming 1.34 release. It's all described in recent threads in
> this newsgroup. If you're interested in Boost 1.33.1 the bcbboost
> project (bcbboost.sourceforge.net) is trying to do the same for the
> current release.

Yes, but the is_base_and_derived patch is not one I had submitted yet
<g>  I would suggest changing BOOST_TESTED_AT to 0x581, rather than
0x570.

--
AlisdairM

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [type_traits] is_base_and_derived on Borland 5.8.1

John Maddock
> Yes, but the is_base_and_derived patch is not one I had submitted yet
> <g>  I would suggest changing BOOST_TESTED_AT to 0x581, rather than
> 0x570.

If you can verify that as the fix, please go ahead and commit,

John.
_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [type_traits] is_base_and_derived on Borland 5.8.1

AlisdairM
John Maddock wrote:

> If you can verify that as the fix, please go ahead and commit,

I will re-confirm that this workaound is still needed with 0x581
tonight, and commit the appropriate version.

--
AlisdairM

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost