Re: [boost] [PropertyMap] proposing compose_property_map

Subject: Re: [boost] [PropertyMap] proposing compose_property_map
From: Jeremiah Willcock (jewillco_at_[hidden])
Date: 2013-02-19 13:26:05

On Sun, 17 Feb 2013, Guillaume Pinot wrote:

> Hi.
>
> Using boost graph, and particulary reverse_graph, I needed a
> compose_property_map adaptor.  As it may be interesting to everyone, I
> propose my code.
>
> http://texitoi.eu/~texitoi/compose_property_map/
>
> I had problems to find a good return type for the get() free function.
> The solution proposed here has the drawback of needing a copy
> constructible type for the value_type of the FPMap in the concept
> checks. I didn't find any other type for the get() function that pass
> the concept checks and that do not return stack variable, and avoid
> useless copy.
>
> The patch may not work completely as I am discovering the boost build
> system.
>
> I am open to any comment.

The one thing I see that needs to be changed is that the test and example 
files need copyright and license notices.  Could you please do that 
update?  Also, I don't remember the policy on tabs in HTML files, but it 
might be nice if you removed the one that was in there.  Thanks.

-- Jeremiah Willcock