Bug 58829

Summary: Clean up printStackTrace() -> proper logging
Product: POI Reporter: Tim Allison <tallison>
Component: POI OverallAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: trivial    
Priority: P2    
Version: 3.14-dev   
Target Milestone: ---   
Hardware: PC   
OS: All   
Bug Depends on:    
Bug Blocks: 58822    

Description Tim Allison 2016-01-08 17:19:24 UTC
On 58822, Jiri Banszel requested we clean up one instance of printStackTrace and convert it to logging via POILogger.

Are there any objections if I clean up the handful of other instances in actual working code?  I'll leave the printStackTraces in test and demo code.
Comment 1 Nick Burch 2016-01-13 07:49:13 UTC
Please feel free to fix this wherever you find it, including tidying up in test/demo code too if appropriate!
Comment 2 Dominik Stadler 2016-01-13 09:54:07 UTC
In r1723899 and r1723960 Andreas replaced all incorrect uses of it in the "production" code and added a forbidden-api-check which prevents newly added ones in prod-code in the future. 

Now only testcases and examples still use it in some places.
Comment 3 Dominik Stadler 2016-01-13 16:26:21 UTC
In r1724445 I removed some more occurrences, I think the remaining ones are mostly intended or test-output, thus I will set this to FIXED, feel free to fix any others that you think are not actually neccessary.