develooper Front page | perl.perl5.porters | Postings from May 2019

Re: [perl #132782] Missing SvPV* utf8/byte nomg macro variants

Thread Previous | Thread Next
From:
pali
Date:
May 17, 2019 14:04
Subject:
Re: [perl #132782] Missing SvPV* utf8/byte nomg macro variants
Message ID:
20190517140437.h3da4i4fyfcor76x@pali
On Thursday 28 March 2019 13:41:30 pali@cpan.org wrote:
> On Tuesday 19 March 2019 20:59:22 Tony Cook via RT wrote:
> > On Tue, 19 Mar 2019 20:49:46 -0700, tonyc wrote:
> > > On Tue, 19 Mar 2019 01:55:18 -0700, pali@cpan.org wrote:
> > > > On Thursday 28 February 2019 15:57:17 pali@cpan.org wrote:
> > > > > In attachment is V5 version of patches. There is updated
> > > > > documentation for them.
> > > > 
> > > > Tony, can you look at new patch version?
> > > 
> > > make test fails.
> 
> Fixed.
> 
> > > There still aren't any vec() tests (see the attached.)
> 
> I included that patch into v6 version.
> 
> > > There's no tests for the new APIs (which can go in ext/XS-APItest/)
> > > 
> > > Tony
> > 
> > Also:
> > 
> > -	    int mg_flags = SV_GMAGIC;
> > +	    int mg_flags = 0;
> > +
> > +            if (flags & SV_GMAGIC)
> > +                mg_flags = SV_GMAGIC;
> > 
> > Can just be:
> > 
> >   int mg_flags = flags & SV_GMAGIC;
> > 
> > but should probably be:
> > 
> >   U32 mg_flags = flags & SV_GMAGIC;
> 
> Changed.
> 
> New version v6 is in attachment.
> 
> Now make test passes. There are some problems with porting/pod_rules.t
> and porting/podcheck.t but I have not touched code for those files... so
> looks like false-positive.

Tony, can you review V6 patches?

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About