You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openjpa.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2013/07/18 23:34:49 UTC

[jira] [Commented] (OPENJPA-2410) Build time detection of System.out/err.print(ln) in source files

    [ https://issues.apache.org/jira/browse/OPENJPA-2410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13712892#comment-13712892 ] 

ASF subversion and git services commented on OPENJPA-2410:
----------------------------------------------------------

Commit 1504673 from [~curtisr7] in branch 'openjpa/trunk'
[ https://svn.apache.org/r1504673 ]

OPENJPA-2410 : Detect SystemOut/Err in code at build time. Patch submitted by Di Wu Lau.
                
> Build time detection of System.out/err.print(ln) in source files
> ----------------------------------------------------------------
>
>                 Key: OPENJPA-2410
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-2410
>             Project: OpenJPA
>          Issue Type: Improvement
>          Components: build / infrastructure
>    Affects Versions: 2.3.0
>            Reporter: Di Wu Lau
>            Assignee: Rick Curtis
>            Priority: Minor
>             Fix For: 2.3.0
>
>         Attachments: OPENJPA-2410.patch
>
>
> Source files may have inadvertent System.out/err.print statements left in source files on commit. Ideally we don't want any of these changes being checked in and a large majority of the time logging facilities should be utilized. That being said, there are a number of cases where we don't have access to a logger and must use System.out/err.
> This JIRA will be used to update our checkstyle to scan for unwanted print statements, and fail the build when/if they are encountered.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira