This was originally posted by Christophe Blin in the JFreeChart patch tracker:
Zip file containing modified sources
Logged In: YES
I decided not to apply this patch, here are the reasons:
- the refactoring of the ChartServlet class into a lot of separate classes is a move in the right direction, but don't think it needs that many classes. You'll see in the 1.1.0 release, that there are now three classes (ChartEngine.java, Parameters.java and DataUtilities.java) that handle the chart creation process. I think these classes are manageable.
- the code uses Java 5 features, but Eastwood is still targetting JRE 1.4.2 and later;
- the custom axis and plot classes were deliberately kept out of the org.jfree.chart.* namespace, because they're generally hacks just to make some of the Google features work. It probably does make sense to move the plot gradient paint over to JFreeChart, but I don't see a good reason to move the axis stuff over;
- the included JUnit tests effectively compare bit map images, which is unlikely to be robust from one JRE to another (particularly across operating systems), so I don't think this is a good approach.
Eastwood Project Leader
Log in to post a comment.