Integrated: 8374208: Remove superfluous field in XYChart
Andy Goryachev
angorya at openjdk.org
Wed Dec 24 22:46:25 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
This pull request has now been integrated.
Changeset: 6591fd4b
Author: Andy Goryachev <angorya at openjdk.org>
URL: https://git.openjdk.org/jfx/commit/6591fd4b9b1bbfedcd9f3dc63bb2e9c5d76bcb6d
Stats: 15 lines in 1 file changed: 8 ins; 2 del; 5 mod
8374208: Remove superfluous field in XYChart
Reviewed-by: arapte, jhendrikx
-------------
PR: https://git.openjdk.org/jfx/pull/2018
More information about the openjfx-dev
mailing list