compiler warnings in VC9

[kfriddile@…]

VC9 at warning level 4 produces the following simple to fix compiler warnings:

line 528 of channel.hpp: Casting the result of the returned expression to the function's return type will silence warnings about conversion to a smaller type.

line 451 of algorithm.hpp: Changing this line to read as follows will silence unused local variable warnings:

void destruct_range_impl(It /*first*/, It /*last*/, mpl::false_) {}

line 586 of algorithm.hpp: Changing this line to read as follows will silence unused local variable warnings:

void default_construct_range_impl(It /*first*/, It /*last*/, mpl::false_) {}

[Hailin Jin]

The last two were fixed in Revision 54727

[mloskot <mateusz@…>]

I think it's preferred to solve this kind of issues using [source:trunk/boost/concept_check.hpp@48821#L45 boost::ignore_unused_variable_warning] sink:

#include <boost/concept_check.hpp>

void destruct_range_impl(It first, It last, mpl::false_)
{
   boost::ignore_unused_variable_warning(first);
   boost::ignore_unused_variable_warning(last);
}

It's also more self-documenting about explicit author's intentions.

[chhenning]

The code changes have a done a while ago. Closing this ticket.