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

Reply via email to