> This PR is a fix for another IOOBE that I discovered while working on #1476. > > The PR simplifies the code for adding a series that already contains data by > adding the data points one-by-one. > As far as I can see no attempt was previously made to optimize the bulk > operation except for some trivial O(1) operations, so this should have no > noticable performance impact. > > Accidentally this fixes another bug related to the missing "negative" style > class when negative data values are added. > > Also, the PR aligns the handling of duplicate categories with the behavior > clarified in #1476, when there are duplicates in the data that was already in > the series before the series was added to the chart. > > Note a change was made to the createTestSeries() test method, letting it > start at index 1, avoiding the duplicate data items resulting from > multiplying by 0. > Without this change `testSeriesRemoveAnimatedStyleClasses` would fail because > it counts the number of plot children, where the duplicates are now removed.
Markus Mack has updated the pull request incrementally with one additional commit since the last revision: fix "negative" style class when series is changed ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1488/files - new: https://git.openjdk.org/jfx/pull/1488/files/a93a2036..4306eb75 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1488&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1488&range=01-02 Stats: 34 lines in 2 files changed: 22 ins; 12 del; 0 mod Patch: https://git.openjdk.org/jfx/pull/1488.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/1488/head:pull/1488 PR: https://git.openjdk.org/jfx/pull/1488
