pdfbox-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hong-Thai Nguyen (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (PDFBOX-2054) Remove System.out.println()
Date Fri, 02 May 2014 21:06:14 GMT

    [ 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)

Mime
View raw message