Re: Result Set Cursor Patch

On Wed, 5 May 2004, Andy Zeneski wrote:

> Kris,
>
> Here it is again, I think my IDE reformatted a few classes which caused
> this patch to be bigger then it needed to be. Here it is in its current
> state.
>
> If the formatting is major problem, I can attempt to reformat it to
> desired settings. I will need to know what the "preferred" way is. I've
> noticed there are several different formats used within these classes,
> I could easily run the tree though Jacobe and clean it up. However,
> that may be too big for the list. :)

The formatting is an issue. Yes, the code is formatted differently all
over the driver, but now is not the time to fix that. There are too many
outstanding patches that will conflict with these whitespace changes to
try and do something about that now. I aim to reindent/reformat at the
start of 7.5 beta when there shouldn't be many outstanding patches or
active development.

Also just trying to read your patch is difficult because it is impossible
to tell if a real change is mixed in with a whitespace change. Could you
resubmit this patch as just your changes.

Kris Jurka