Summary: | [PATCH] Replace System.err calls with Logger call | ||
---|---|---|---|
Product: | POI | Reporter: | Torsten Krah <tkrah> |
Component: | POI Overall | Assignee: | POI Developers List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | rpi_alum |
Priority: | P2 | ||
Version: | 3.9-dev | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Linux | ||
Attachments: |
Patch against rev 1404603
Patch against rev 1406238 |
Description
Torsten Krah
2012-02-09 12:02:20 UTC
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.
|