javafx - OpenJFX 9-dev XYChart.java...is this a bug? -
i have been browsing through source code relating javafx charts lately, , came across following method:
/** * should called seriesremoved() when finished animation deleting series * chart. remove series showing in iterator returned getdisplayedseriesiterator(). * * @param series series remove */ protected final void removeseriesfromdisplay(series<x, y> series) { if (series != null) series.settoremove = false; series.setchart(null); displayedseries.remove(series); } if series passed in null, there nothing keep making call series.setchart(null); will, if series null, cause nullpointerexception. given method protected , therefore visible sub-classes, shouldn't if statement contain brackets surround entire method? essentially:
/** * should called seriesremoved() when finished animation deleting series * chart. remove series showing in iterator returned getdisplayedseriesiterator(). * * @param series series remove */ protected final void removeseriesfromdisplay(series<x, y> series) { if (series != null) { series.settoremove = false; series.setchart(null); } displayedseries.remove(series); } given openjfx authors better programmers am, hesitant report bug, given not setup (via jira system or mailing lists, wanted ask more knowledgeable folks me before proceed embarrass myself!)
Comments
Post a Comment