Re: [boost] [1.35.0] Test regressions

From: Beman Dawes (bdawes_at_[hidden])
Date: 2008-03-19 11:58:12

Doug Gregor wrote:
> On Mar 19, 2008, at 9:42 AM, Beman Dawes wrote:
> 
>> For a full list of 1.35.0 test regressions, see
>> http://beta.boost.org/development/tests/release/developer/summary.html
> 
> Or, better yet:
> 
> 	http://beta.boost.org/development/tests/release/developer/issues.html
> 
>> There are a few 1.35.0 regressions I'm not particularly concerned
>> about;  VC++ 7.1 is now old news, and no one seems to care much about
>> the Intel/Windows compiler.
>>
>> Given that VC++ 9.0 has been out for awhile now, it would be nice to
>> kill off a few more of those regressions before 1.35.0 ships:
>>
>> * Graph
> 
> As Richard Webb noted, it fails for the same reason as VC++ 8.0 fails,  
> so we should just mark this as "expected" as well. Okay for me to go  
> ahead and do that on the trunk and release branch?

Yes, please!

> 
>> Red regressions I'm concerned about are:
>>
>> * Numeric/interval on Darwin.
>> * Serialization on Darwin.
> 
> This is the trivial patch to fix the Serialization failures on Darwin:
> 
> Index: libs/serialization/example/portable_binary_oarchive.hpp
> ===================================================================
> --- libs/serialization/example/portable_binary_oarchive.hpp	(revision  
> 43724)
> +++ libs/serialization/example/portable_binary_oarchive.hpp	(working  
> copy)
> @@ -52,7 +52,7 @@ public:
>       friend class  
> boost::archive::basic_binary_oarchive<portable_binary_oarchive>;
>       friend class boost::archive::save_access;
>   #endif
> -    void save_impl(const long l){
> +    void save_impl(long l){
>           long ll = l;
>           char size = 0;
>           if(l < 0){
> 
> Tested on Darwin (both PowerPC and Intel); shall I commit to the trunk  
> and the release branch?

Yes, thanks!

--Beman