Re: Clean up wasNullFlag usage

[Kris Jurka <books(at)ejurka(dot)com>]

On Sat, 21 Jul 2007, Mikko Tiihonen wrote:

> The wasNullFlag must be evaluated for each getXXX method invocation.
> Currently it is done all over the ResultSet classes. The patch moves
> the evaluation inside checkResultSet method that is already invoked at
> the beginning of each getXXX method. Also updates the javadoc to be
> explicit about the functionality.

One of the reasons for doing the ad-hoc null flag setting was to avoid 
multiple checkResultSet calls like you've now introduced for things like 
getByte.  Rethinking that decision now, the performance impact of 
checkResultSet is virtually non-existent, so this cleanup makes sense.

Applied with some additional minor modifications (missed 
Jdbc[24]ResultSet, mention the fact that getFastXXX can no longer handle 
null values).

I don't like slipping in unrelated functionality changes like this one, 
which I took out:

+        if (fields[columnIndex - 1].getOID() == Oid.BOOL) {
+            return toBoolean(getFixedString(columnIndex)) ? 1 : 0;
+        }

I'm not opposed to the idea, but please raise things like this as separate 
issues and not silently lump it in with a change that supposedly didn't 
affect functionality.  Please resubmit.

Kris Jurka