Bug 60805

Summary: Remove calls to System.out.print and System.out.println
Product: POI Reporter: Javen O'Neal <onealj>
Component: POI OverallAssignee: POI Developers List <dev>
Status: NEW ---    
Severity: enhancement CC: tallison
Priority: P2    
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: All   
Bug Depends on: 60787    
Bug Blocks:    

Description Javen O'Neal 2017-03-02 19:21:29 UTC
System.out.print and System.out.println could potentially be added to forbidden apis if we can exclude dev classes.

grep -r -l --exclude-dir=.svn System.out.println src/java/ src/ooxml/java src/scratchpad/src/
lists 62 files, most of which are dev classes, but includes some classes that shouldn't be using System.out.println, such as:
src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java
Comment 1 Dominik Stadler 2017-03-19 21:09:07 UTC
There is a bundled signature for this:

            <bundledsignatures name="jdk-system-out"/>


However disabling forbidden-apis in all the examples and devtools via annotation is a bit tedious and excludes all api-checks in those classes/methods.
Comment 2 Javen O'Neal 2017-09-19 15:21:47 UTC
*** Bug 61536 has been marked as a duplicate of this bug. ***
Comment 3 Tim Allison 2017-09-19 15:24:36 UTC
Doh.  Thank you, Javen...
Comment 4 Tim Allison 2017-09-19 15:56:32 UTC
r1808903 focused on println in tests.

* redirected stdout for x.main(args) calls.

* deleted a few printlns that were prob useful during development.

* commented out a few printlns that I felt someone might want to turn back on during development

* converted some to POILogger.log()

* left the printlns in MemoryUsage...not sure what to do with those?!