On Wed, Jan 11, 2006 at 03:29:03AM -0800, Gisle Aas wrote:
> Nicholas Clark <[EMAIL PROTECTED]> writes:
> > --- perl/mg.c#93~26689~ 2006-01-06 15:03:51.000000000 -0800
> > +++ perl/mg.c 2006-01-10 02:44:50.000000000 -0800
> > @@ -569,6 +569,13 @@
> > return 0;
> > }
> >
> > +#define SvRTRIM(sv) STMT_START { \
> > + STRLEN len = SvCUR(sv); \
> > + while (len > 0 && isSPACE(SvPVX(sv)[len-1])) \
> > + --len; \
> > + SvCUR_set(sv, len); \
>
> This ought to also NUL terminate the string:
>
> *SvEND(sv) = '\0';
Ooh yes, well spotted. Fixed with change 26781
Nicholas Clark
