Bryan Pendleton
added a comment - 01/Jul/12 22:43 I am reviewing this patch.
I looked at the svn history of NetStatementReply.java
http://svn.apache.org/viewvc/db/derby/code/trunk/java/client/org/apache/derby/client/net/NetStatementReply.java?view=annotate
and all of these methods seem to have been present since
the original contribution of this class to Derby, and have not been
modified since.
I also independently searched the code, and I agree that these
methods are entirely unused.
I'm intending to commit this patch. Deleting this unused code seems
fine, since the original implementation is still preserved in svn in case
we want to revive any of these methods in the future.