polygon_*_set_data::get() does not behave according to the documentation about polygon

[asydorchuk]

Hi Ton, I was not present during initial development of the library, so not sure what was the actual intent. However, you are right, that the implementation is inconsistent with documentation.

[ton]

Hi asydorchuk, thanks for the feedback. Do you know whether this is something that used to work? I can investigate this a bit further once I come around to it, but it will take some time since I am completely unfamiliar with the code.

In any case, it is not a critical issue, since it is pretty easy to workaround, at the cost of some runtime performance. So in case this can be more efficiently handled on the Boost Polygon side, that would be great.

[asydorchuk]

Hi Ton, it's a bit hard to say what is the right thing to do in this situation. Since updating the code will make it backward incompatible for people who already built workarounds on their side. Perhaps the best solution is to update the documentation according to the current behaviour. What do you say?

[ton]

Hey asydorchuk, I agree changing behavior might be unwanted for existing users, so then only changing the documentation is required indeed.