Boost logo

Boost :

Subject: Re: [boost] [1.46] Last few merges needed?
From: Joachim Faulhaber (afojgo_at_[hidden])
Date: 2011-01-29 13:38:45


Dear release managers,

as discussed before, I've done my homework related the bug that I've
detected, which is described in ticket #5135:
https://svn.boost.org/trac/boost/ticket/5135. See details below.

2011/1/27 Joachim Faulhaber <afojgo_at_[hidden]>:
> 2011/1/27 Rene Rivera <grafikrobot_at_[hidden]>:
>> On 1/27/2011 10:48 AM, Joachim Faulhaber wrote:
>>> Such last minutes surprises are always not very welcome, I know ... I
>>> won't take your decision personal ;)
>>
>> After a brief talk on IRC.. How about if you..
>>
>> 1. Add a test to show the problem in trunk. Passing with the fix in it of
>> course.

1.1: I've fixed the code (see patchfile)
Index: is_interval.hpp
===================================================================
--- is_interval.hpp (revision 68546)
+++ is_interval.hpp (working copy)
@@ -26,7 +26,7 @@
 {
     typedef is_interval<Type> type;
     BOOST_STATIC_CONSTANT(bool,
- value = (interval_bound_type<Type>::value <
interval_bounds::undefined));
+ value = ((interval_bound_type<Type>::value) <
interval_bounds::undefined));
 };

@@ -34,7 +34,7 @@
 {
     typedef has_static_bounds<Type> type;
     BOOST_STATIC_CONSTANT(bool,
- value = (interval_bound_type<Type>::value <
interval_bounds::dynamic));
+ value = ((interval_bound_type<Type>::value) <
interval_bounds::dynamic));
 };

 template <class Type> struct has_dynamic_bounds
===================================================================

1.2 I've added two test to the trunk that test the bug fix:

(1.2.1) A minimal test containing the bug and it's fix
http://svn.boost.org/svn/boost/trunk/libs/icl/test/fix_include_after_thread_/fix_include_after_thread.cpp

(1.2.2) A test with includes of ICL-files that caused the error before
when preceeded by boost/thread.hpp
http://svn.boost.org/svn/boost/trunk/libs/icl/test/fix_icl_after_thread_/fix_icl_after_thread.cpp

The tests ran successfully for 2 nights.

>> 2. Wait for the beta to get done.
Has happened tonight.

>> 3. Then you can ask again to make this change after the beta is out. And
>> we'll likely say yes.

I have prepared a merge that does the fix and adds the two new tests.
Do I have permission now to commit the merge to the release branch?

Best regards,
Joachim


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk