Re: [boost] [Review Request] Inclusion of the Boost.Polygon Voronoi Library

Subject: Re: [boost] [Review Request] Inclusion of the Boost.Polygon Voronoi Library
From: Andrii Sydorchuk (sydorchuk.andriy_at_[hidden])
Date: 2012-05-22 13:37:57

>
> The objection to void* is a style objection.  I think if you made the
> field an int (preferably std::size_t ) then you could name it color instead
> of data and it would be clear it was intended for graph algorithms.  As a
> std::size_t it would be usable as an index into a vector of arbitrary type
> to allow complex data to be associated with the graph elements in O[1] time
> for more complex use cases.  That way you preserve the ability to implement
> graph algorithms directly on the data structure and can update your example
> without needing to declare your own classes.

This actually sounds pretty good to me.

So now we have two possible solutions:
1) removing data field member completely;
2) substitute it with size_t color member.

I am satisfied with both solutions (the second one is more flexible as it
allows to execute graph algorithms directly).

Regards,
Andrii

_______________________________________________
Unsubscribe & other changes:
http://lists.boost.org/mailman/listinfo.cgi/boost