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

Regards,
Andrii

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


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk