On Wed, Sep 03, 2003 at 11:10:33AM -0400, Tom Lane wrote:
> Karel Zak <[EMAIL PROTECTED]> writes:
> >  Fixed. All tests  passed. My court  is without  ball now. The  patch is
> >  attached.
> 
> Applied to CVS HEAD.  I also applied just the
> 
> +             if (flag == FROM_CHAR && *s=='\0')
> +                     break;
> 
> part to the 7.3 branch --- the rest of the changes didn't apply cleanly,
> and seemed unrelated anyway.  This fixes the immediate complaint, but

 Hmm..  but  the patch  fix  another  small  bug  which I  found  today.
 Please,  if  you don't  submit  all  of the  patch,  fix  two lines  in
 do_to_timestamp():
 
 line: 2997 must be: 
    incache = TRUE;

 line: 3039 must be: 
    if (!incache)

 (Note:  this another  bug appear  if  format picture  for date/time  is
 greater than 128 bytes -- for this probably nobody find it yet)
 
> if there's some reason to apply the rest to 7.3.*, would you submit
> an additional patch against that branch?

 Yep.

    Karel

-- 
 Karel Zak  <[EMAIL PROTECTED]>
 http://home.zf.jcu.cz/~zakkr/

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
    (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to