Boost logo

Boost :

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).


Unsubscribe & other changes:

Boost list run by bdawes at, gregod at, cpdaniel at, john at