On Sat, Oct 15, 2005 at 11:02:42PM -0400, Eric Lenio wrote:
> On Fri, Oct 14, 2005 at 10:09:17PM +0100, Tim Bunce wrote:
> > 
> > Here's an off-the-top-of-my-head patch, totally untested and barely
> > thought about, but it seems plausible...
> > 
> > Index: oci8.c
> > ===================================================================
> > --- oci8.c      (revision 1086)
> > +++ oci8.c      (working copy)
> > @@ -1223,9 +1223,9 @@
> >                 /* FALLTHRU */
> >         case  96:                               /* CHAR         */
> >                 fbh->disize = fbh->dbsize;
> > +               fbh->prec   = fbh->disize;
> >                 if (CS_IS_UTF8(fbh->csid)) 
> >                     fbh->disize = fbh->dbsize * 4;
> > -               fbh->prec   = fbh->disize;
> >                 break;
> >         case  23:                               /* RAW          */
> >                 fbh->disize = fbh->dbsize * 2;
> > 
> > Let me know if the test suite runs clean with this.
> 
> Tim yes the test suite does run clean, including using the additional
> test I had made for t/20select.t.  Do you need any further information
> from me on this?

No, thanks, but if you've got some time I'd appreciate more patches to
extend the test suite further...

Tim.

Reply via email to