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