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