Hi,
I'm in the process of working on a J2EE project that is using geotools quite extensively. Geotools is great, but there appears to be a somewhat irritating use of System.out.print* and System.err.print* peppered throughout the code (207 and 43 occurrences respectively in the 2.2.trunk tree).
Surely, it is better to use only one logging mechanism as this can be effectively re-directed and controlled (to be fair, there are 2602 references to this in the source tree). The use of System.* completely messes up my stdout logs & causes a huge amount of unneccessary logfiles in a production system.
I would go through and alter the relevant statements, but they are peppered widely throughout the source and I do not have write access to the svn repository. I would suggest that module maintainers do this as part of the release process.
FYI, the searches I carried out are:
grep -r System.out * | grep -v test | grep -v "//" | grep -v ".svn" | grep -v spike | grep -v scripts | grep -v javadoc | grep -v demo | wc -l
grep -r System.err * | grep -v test | grep -v "//" | grep -v ".svn" | grep -v spike | grep -v scripts | wc -l
grep -r LOGGER * | grep -v Logger.getLogger
Kind regards and thanks for all the hard work,
John Grange
Senior Software Engineer
Tel: +44 (0)1749 834922
email: [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>
BlueFinger Limited
Underwood Business Park
Wookey Hole Road, WELLS. BA5 1AF
Tel: +44 (0)1749 834900
Fax: +44 (0)1749 834901
web: www.bluefinger.com
Company Reg No: 4209395 Underwood Business Park, Wookey Hole Road, Wells, Somerset BA5 1AF.
*** This E-mail contains confidential information for the addressee only. If you are not the intended recipient, please notify us immediately. You should not use, disclose, distribute or copy this communication if received in error. No binding contract will result from this e-mail until such time as a written document is signed on behalf of the company. BlueFinger Limited cannot accept responsibility for the completeness or accuracy of this message as it has been transmitted over public networks.***
