[Boost-bugs] [Boost C++ Libraries] #2594: Bub into all_reduce

Subject: [Boost-bugs] [Boost C++ Libraries] #2594: Bub into all_reduce
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2008-12-16 15:56:52

#2594: Bub into all_reduce
------------------------------------+---------------------------------------
 Reporter:  micdestefano_at_[hidden]  |       Owner:  dgregor
     Type:  Bugs                    |      Status:  new    
Milestone:  Boost 1.38.0            |   Component:  mpi    
  Version:  Boost 1.36.0            |    Severity:  Problem
 Keywords:  all_reduce,             |  
------------------------------------+---------------------------------------
 Good Morning.[[BR]]

 I've found a bug into the file[[BR]][[BR]]
 `boost/mpi/collectives/all_reduce.hpp`[[BR]][[BR]]
 At line 36 you should change[[BR]][[BR]]
 `is_mpi_op<Op, T>::op()`[[BR]]
 [[BR]]
 into[[BR]]
 [[BR]]
 `(is_mpi_op<Op, T>::op())`[[BR]]
 [[BR]]
 i.e. add a couple of parentheses around the expression. Without those
 parentheses, this build error occurs:[[BR]]
 [[BR]]
 `error: macro "MPI_Allreduce" passed 7 arguments, but takes just 6`[[BR]]
 [[BR]]
 because the comma that separates template parameters confuses the
 preprocessor.[[BR]]
 [[BR]]
 I've noticed that the bug is still present into the Boost.MPI development
 version.[[BR]]
 [[BR]]
 My compiler is gcc 4.1.2.[[BR]]
 [[BR]]
 Thank you in advance.[[BR]]
 Michele De Stefano

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/2594>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.