Re: [boost] [BGL] container_gen

Subject: Re: [boost] [BGL] container_gen
From: Gordon Woodhull (gordon_at_[hidden])
Date: 2011-09-21 13:36:45

Hi Cromwell,

On Sep 21, 2011, at 11:40 AM, Cromwell Enage wrote:
> After a bit of a programming hiatus, I've managed to follow up on our container_gen discussion we had last May, archived here: <http://article.gmane.org/gmane.comp.lib.boost.devel/219238>

Sorry for the slow response - I've been in transit and starting an exciting new job.

> I've created a branch in the sandbox called utility-container_gen that 
> places the metafunction in Boost.Utility, changes 
> detail::is_random_access to is_random_access_selector, provides an 
> is_unique_associative_selector metafunction, and changes adjacency_list 
> and parallel_edge_traits to use said utilities.  Plus, there's Quickbook documentation.

This looks really great.  It's wonderful have this properly documented.

My only concern about the documentation is that it jumps without warning into Tree documentation, which is confusing.  [I'm dying to talk with you about the tree stuff, because I can't really do Fusion.Graph without also considering Fusion.Tree --- but I've got a lot of other stuff on my plate, so it'll have to wait a little while.]

Also the documentation says there is a dependency on type traits operators, but doesn't say why.  I mean to investigate this myself, but perhaps you should also state why in the doc.

> I've tested a few example programs in the BGL to make sure they still 
> work, but I feel that's far from being thorough.  No selectors for 
> Boost.Intrusive have been implemented, yet.

As I mentioned before, Boost.Intrusive selectors would also have to generate some data to be put into the individual vertices/edges.  I don't know yet whether it will be easy to add these to BGL objects.

Cheers,
Gordon