Re: Result Set Cursor Patch

Or add a -w to your diff to ignore whitespace differences.

--Barry

Kris Jurka wrote:
>
> 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
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: the planner will ignore your desire to choose an index scan if your
> joining column's datatypes do not match