Subject | Re: 3 new bugs on source forge |
---|---|
Author | AlwaysLinux2 |
Post date | 2002-08-26T22:23:21Z |
Hmm. that would make sense, but FBField.isNull()'s current
definition is :
return (field.sqlind== -1);
(line 157 of FBField.java)
I guess I could have fixed that line instead to something like
return (field.sqldata == null);
but then I wouldn't have been sure (short of a comprehensive code
review) that nothing else looked at sqlind, so I would still have
done the fix in GDS_Impl, just to be on the safe side.
Jeremy Williams
definition is :
return (field.sqlind== -1);
(line 157 of FBField.java)
I guess I could have fixed that line instead to something like
return (field.sqldata == null);
but then I wouldn't have been sure (short of a comprehensive code
review) that nothing else looked at sqlind, so I would still have
done the fix in GDS_Impl, just to be on the safe side.
Jeremy Williams
--- In Firebird-Java@y..., David Jencks <davidjencks@d...> wrote:
> Thanks, I hope to look at this tonight.
>
> I will have to look hard at the sqlind situation, since the last
time I
> worked on that code I thought I modified everything to decide if a
field
> was null based on whether the object holding the field was null.
If I
> can't sort out the situation I will just apply your patch.
>
> Many thanks!
>
> david jencks
>