[signals] bug in named_slot_map.hpp ?

Dear boosters, line 103 of /cvsroot/boost/boost/boost/signals/detail/named_slot_map.hpp I believe it should be last_group(last) instead of last_group(group) If it is not the case then the argument last should be commented out to avoid "unused" warnings Best regards C. Index: named_slot_map.hpp =================================================================== RCS file: /cvsroot/boost/boost/boost/signals/detail/named_slot_map.hpp,v retrieving revision 1.8 diff -u -r1.8 named_slot_map.hpp --- named_slot_map.hpp 19 Apr 2005 13:50:53 -0000 1.8 +++ named_slot_map.hpp 25 May 2005 10:07:39 -0000 @@ -100,7 +100,7 @@ { init_next_group(); } named_slot_map_iterator(group_iterator group, group_iterator last, slot_pair_iterator slot) : - group(group), last_group(group), slot_(slot), slot_assigned(true) + group(group), last_group(last), slot_(slot), slot_assigned(true) { } void init_next_group() -- MIT Affiliate

Doug Gregor <dgregor@cs.indiana.edu> writes:
On May 25, 2005, at 5:09 AM, Christophe Prud'homme wrote:
Dear boosters,
line 103 of /cvsroot/boost/boost/boost/signals/detail/named_slot_map.hpp I believe it should be last_group(last) instead of last_group(group)
Yep, that's a bug. Fixed, and thanks!
Doug
What can we add to the regression tests that would have caught the problem? Robert Zeh
participants (3)
-
Christophe Prud'homme
-
Doug Gregor
-
Robert Zeh