RFR: 8374208: Remove superfluous field in XYChart
Ambarish Rapte
arapte at openjdk.org
Wed Dec 24 01:26:07 UTC 2025
On Tue, 23 Dec 2025 20:01:22 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
> This change removes an unnecessary field `XYChart.Data.series` as it is mirrored by the `seriesProperty`, something I failed to see twice - first during code review [0] I did not realize a property was added that mirrors the field, then I wrongly thought it was added in [1].
>
> Now is the time to gather stones.
>
> [0] https://git.openjdk.org/jfx/pull/1016
>
> [1] https://git.openjdk.org/jfx/pull/2013
LGTM...
-------------
Marked as reviewed by arapte (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/2018#pullrequestreview-3609658446
More information about the openjfx-dev
mailing list