#5 Eastwood Rewrite


This was originally posted by Christophe Blin in the JFreeChart patch tracker:



  • David Gilbert

    David Gilbert - 2008-06-27

    Zip file containing modified sources

  • David Gilbert

    David Gilbert - 2008-07-18
    • assigned_to: nobody --> mungady
    • status: open --> closed-rejected
  • David Gilbert

    David Gilbert - 2008-07-18

    Logged In: YES
    Originator: 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.


    Dave Gilbert
    Eastwood Project Leader


Log in to post a comment.