I am reworking my combined domain and range plot classes (see https://sourceforge.net/tracker/?func=detail&atid=315494&aid=1924543&group_id=15494 (patch 1924543)), and I have noticed a discrepancy between the CombinedDomainCategoryPlot and CombinedDomainXYPlot. In the latter, the getDataRange(ValueAxis) method has been defined to return the combined rnage of all the subplots. In the former, the getDataRange(ValueAxis) method is not defined, so the CombinedDomainCategoryPlot relies on the CategoryPlot class' version. The result is that calling getDataRange on a CombinedDomainCategoryPlot returns a default axis range of [0.0, 1.05].
This discrepancy can be fixed by either adding CombinedDomainXYPlot's getDataRange(ValueAxis), appropriately modified, to CombinedDomainCategoryPlot or remove the override from CombinedDomainXYPlot. While I do not have strong feelings as to which course of action should be taken, I do feel strongly that this should be fixed. The argument for the former solution is backwards compatibility in case people have relied on CombinedDomainXYPlot's getDataRange(ValueAxis). The argument for the latter solution is 'correctness' since the combined plot does not really have a data range per se.
I will be updating the patch with my combined domain and range plot classes, and I will be noting this bug in the comments.
Log in to post a comment.