Remove all System.out (replace by CORRECT logging) - and ensure they don't come 
back
------------------------------------------------------------------------------------

                 Key: MIFOS-3923
                 URL: http://mifosforge.jira.com/browse/MIFOS-3923
             Project: mifos
          Issue Type: Bug
            Reporter: Michael Vorburger


I've found 69x System.out, which I'd argue are bugs and should be removed and 
replaced by CORRECT logging.  System.out is a performance bottleneck (as it, 
usually, synchronizes on the console), not relevant in single user tests & dev, 
but measurable in production / load scenarios.

In tests they are less of a problem, in src/main they should be absolutely 
forbidden.

There are also 12 System.err - I had a quick look through them and they are all 
in test or helpers (in src/main) which are probably called from tools main() 
and not running Web app code, so may be fine?

Ideally this issue should include some action to ensure they don't come back... 
e.g. make sure your shared FindBugs or Checkstyle config (if you do this?) are 
configured to flag it up red -  or just fail the build if any are 
re-introduced, that usually helps ;).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://mifosforge.jira.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today.
http://p.sf.net/sfu/beautyoftheweb
_______________________________________________
Mifos-issues mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mifos-issues

Reply via email to