[
https://issues.apache.org/jira/browse/PDFBOX-2054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13987830#comment-13987830
]
Hong-Thai Nguyen edited comment on PDFBOX-2054 at 5/2/14 9:04 PM:
------------------------------------------------------------------
Yes, we use PDFbox via Tika in many connectors. We direct out & err separately
in logs files, and log4j in other file with something like this:
java -Dlog4j.configuration=file:log4j.properties -cp CP_PATH runClass
^>%LOG_FOLDER%\out.txt 2^>%LOG_FOLDER%\err.txt
Normally, we expect that all log must be group in log4j (with configuration on
logger, level ...), stdout (out.txt) and errou(err.txt) are generally reserved
for console things.
Otherwise, printStackTrace() & out.println() has nothing about date/time and
more less uncontrollable.
was (Author: thaichat04):
Yes, we use PDFbox via Tika in many connectors. We direct out & err separately
in logs files, and log4j in other file with something like this:
java -Dlog4j.configuration=file:log4j.properties -cp CP_PATH runClass
^>%LOG_FOLDER%\out.txt 2^>%LOG_FOLDER%\err.txt
Normally, we expect that all log must be group in log4j (with configuration on
logger, level ...), out.txt and err.txt are reserved for something else.
Otherwise, printStackTrace() & out.println() mentions nothing about date/time
and more less controllable.
> Remove System.out.println()
> ---------------------------
>
> Key: PDFBOX-2054
> URL: https://issues.apache.org/jira/browse/PDFBOX-2054
> Project: PDFBox
> Issue Type: Bug
> Affects Versions: 1.8.4, 1.8.5, 1.8.6, 2.0.0
> Reporter: Hong-Thai Nguyen
> Priority: Minor
>
> For example at GlyfSimpleDescript.java
> {code}
> ...
> catch (ArrayIndexOutOfBoundsException e)
> {
> System.out.println("error: array index out of bounds");
> }
> {code}
> and also 'printStackTrace' like in PageDrawer.java:
> {code}
> ...
> catch( IOException io )
> {
> io.printStackTrace();
> }
> {code}
> Should forward exception or keep silence.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
