It seems really weird to me to see code like: os = NullOutputStream.NULL_OUTPUT_STREAM;
instead of: os = NullOutputStream.INSTANCE; or: os = NullOutputStream.getInstance(); Any objection to this change? Gary -- E-Mail: garydgreg...@gmail.com | ggreg...@apache.org Java Persistence with Hibernate, Second Edition <http://www.manning.com/bauer3/> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> Spring Batch in Action <http://www.manning.com/templier/> Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory