That might work for me, I'm not sure. Here's more background. There is much I want to do with the charts API already, but I haven't had time to see if this change goes in directions I'm wanting already. In particular I am constantly having to do variations on "instanceof" and type casting and/or use reflection to retrieve elements common to most or all chart types but defined without a common interface or type hierarchy. The current design follows the OOXML definitions, which have much overlap and duplication but don't lend themselves to identifying common attributes. I've started doing some of that with a parent class for axes, but there needs to be more. This XDDF work could help that, and at worst would likely only move the work to a different package, not add to it.
Delegating the public API, however, may break the necessary introspection and accessing of CT* classes done currently by both my own project and Vaadin-Spreadsheet-Charts. That would be a serious problem, as I'm having to walk a very fine line already with them - they don't dedicate a lot of resources, and I'm having to do all the heavy lifting, but I'm a one-person operation, in charge of all of development and IT for my employer. The project using POI is only supposed to be half my time :) Without testing it, I don't know yet how extensive the changes would be and how much backward compatibility would be broken, especially given that some of that compatibility uses reflection to access fields not necessarily already made public, because not all chart features are accessible from the POI API. On Fri, Aug 11, 2017 at 9:16 AM Javen O'Neal <[email protected]> wrote: > That sounds like a good compromise. > > On Aug 11, 2017 04:18, "pj.fanning" <[email protected]> wrote: > > > Would it be feasible to keep the existing classes and APIs and have them > > delegate to the XDDF impl? > > We could immediately deprecate any of these legacy APIs. > > > > > > > > -- > > View this message in context: http://apache-poi.1045710.n5. > > nabble.com/XDDF-implementation-shared-between-XSSFChart-and-XSLFChart- > > tp5728473p5728476.html > > Sent from the POI - Dev mailing list archive at Nabble.com. > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > >
