This is more a enhancement / feature request - did not found any task yet for this one, maybe its a duplicate. There are some System.err calls in the Poi code, e.g. Sheet.java in Scratchpad. May it be possible that these call are replaced by a call to $logger.error(...) instead to integrate them in the logging system a user may choose?
Would it be possible for you to work up a patch to do this change?
Yes i can do this patch and submit it. Is it enough to do this against latest trunk?
Patch question: Would you like to have a logger per name or a logger per class, e.g. getLogger("poi-error") and all system.err calls use this one, or should it be a getLogger(XXX.class) one?
getLogger(XXX.class) should be fine. Yegor (In reply to comment #3) > Patch question: Would you like to have a logger per name or a logger per class, > e.g. getLogger("poi-error") and all system.err calls use this one, or should it > be a getLogger(XXX.class) one?
A patch file has been supplied for fixing this issue by replacing most instances of System.err with a POILogFactory.getLogger() logger. SerialVersionUID's and proper Argument types were applied to lists. The most complex changes were applied to the HPSFRuntimeException() class where the print stack trace logic was removed where System.err was being explicitly declared. By commenting this out, the base RuntimeException printStackTrace logic should prevail. The soon to be added patch file was applied to the current POI daily trunk 3.9-beta1 on November 1st, 2012. Revision (1404603)
Created attachment 29540 [details] Patch against rev 1404603
For some reason scracthpad was not imported into eclipse when I imported from the ant build file. I ended up missing the System.err calls from that project. A newer more complete patch file is being attached shortly that uses loggers for scratchpad related code as well. This will replace the prior patch file.
Created attachment 29561 [details] Patch against rev 1406238 Replace System.err info messages with a POILogger.
Thanks for the patch, applied in r1410318 Yegor