Re: [boost] [interval] Borland patch

From: Guillaume Melquiond (guillaume.melquiond_at_[hidden])
Date: 2005-06-17 06:54:53

Hi,

I'm currently traveling, so I only have access to the Internet
intermittently. My replies may be delayed. Sorry.

Le mardi 14 juin 2005 à 07:15 +0000, AlisdairM a écrit :
> The interval library is currently marked 'unusable' for Borland so any
> patch will not show up in regular regression testing!

The rule for previous Boost releases was to mark failing tests either as
"expected failures" or as "unusable libraries". Since it didn't make
much sense to me to mark them as "expected", I marked them as
"unusable", but the drawback is that they indeed don't show in
regression testing anymore.

> There is a strange compile error that the member functions lower/upper
> cannot be found in class interval.  I still don't have a small test
> case for the bug, but it appears to depend on the order of declaration
> in the class itself.  Moving these two declarations higher up the file
> solves the problem:

Thanks, committed.

> This assumes the library author wants to start supporting Borland again
> of course ;¬)

I never got Borland free tools (and bjam) to work correctly under Wine,
so a real support of Borland compiler is a bit out of my reach. However
I will gladly accept any patch that makes the library works with this
compiler.

How are the tests with your patch applied? Are they clear? If they are,
I will remove the "unusable" flag so that the compiler appears in the
regression logs again.

Best regards,

Guillaume