Dear all,

Since June, this is the third pull request I open about the same theme.

After the first review, mentioning a conversation I had missed 
(https://lists.apache.org/thread.html/ff5470bc413d298188f8c5d3d250ee15d36bd655227696869ed21aab@%3Cdev.poi.apache.org%3E),
 I reworked my initial submission to refactor my code into some sharable XDDF 
package for the common DrawingML subset about charts.

After I had opened the second pull request, I wanted to solve some personal 
situation which my initial XDDF implementation did not allow me to get done, 
not even with ugly code. So I decided to go for some more steps, learning from 
the XSSF charts implementation, and migrating the existing XSSF charts and 
tests to use the new XDDF code, which evolved a lot.

This is why we now have pull request #68 (https://github.com/apache/poi/pull/68 
<https://github.com/apache/poi/pull/68>) and pull requests #61 and #67 have 
been closed.

Personally, I was scratching my own itch which was bug #57835 
(https://bz.apache.org/bugzilla/show_bug.cgi?id=57835), at least the Charts 
part of the bug report. Maybe the Notes part could be solved following some 
strategy similar to what I implemented for Charts. It did not enter my scope 
yet. Anyway, it would be worth its own pull request, since this one is already 
touching 97 files…

I hope this contribution does not fall in the middle of some important release 
preparation.

Best regards,
Alain


Reply via email to