On Fri, 9 Aug 2024 15:37:56 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> `reconstruct` is a misnomer, as reconstruction is a two-step operation >> (`convertBack` followed by `convert`). To be fair, I don't like any of these >> better than `convertBack`, mostly because I want to stress that those two >> operations are closely related, and none of the alternatives evoke that >> sentiment. That being said, this is a minor API and I don't really mind >> either way, but absent a compelling reason, I'd prefer to stick to >> `convertBack`. > > That's fine, I just think `convertBack` implies an arrow of time or a > preferred spatial direction. > > Maybe `toMap()`, but as long as the method is sufficiently documented, we > should be fine. Naming _is_ hard. I've moved this to an internal interface (it doesn't need to be public for this PR), so we can have the discussion later if we choose to make this API public (and if so, how we go about doing that). ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1522#discussion_r1712445361
