Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5.22.2-RC1 & 5.24.0-RC1: Extra slash in perldelta example #15276

Closed
p5pRT opened this issue Apr 14, 2016 · 7 comments
Closed

5.22.2-RC1 & 5.24.0-RC1: Extra slash in perldelta example #15276

p5pRT opened this issue Apr 14, 2016 · 7 comments

Comments

@p5pRT
Copy link

p5pRT commented Apr 14, 2016

Migrated from rt.perl.org#127899 (status was 'resolved')

Searchable as RT127899$

@p5pRT
Copy link
Author

p5pRT commented Apr 14, 2016

From @trwyant

Created by @trwyant

Perldelta for 5.24.1-RC1 and 5.22.2-RC1 contain, under "Selected Bug
Fixes", the item

  * Certain regex patterns involving a complemented POSIX class in an
  inverted bracketed character class, and matching something else
  optionally would improperly fail to match. An example of one that
  could fail is "/qr/_?[^\Wbar]\x{100}/". This has been fixed. [perl
  #127537]

It looks to me like the leading slash in the example given on the last
full line is spurious. Or maybe the first three characters "/qr". At any
rate, the intent of the author is unclear, at least to this reader.

Thank you very much for Perl 5.24.0 and 5.22.2.

Perl Info

Flags:
    category=docs
    severity=low

Site configuration information for perl 5.24.0:

Configured by tom at Thu Apr 14 09:17:45 EDT 2016.

Summary of my perl5 (revision 5 version 24 subversion 0) configuration:
   
  Platform:
    osname=darwin, osvers=15.4.0, archname=darwin-2level
    uname='darwin samwise.local 15.4.0 darwin kernel version 15.4.0: fri feb 26 22:08:05 pst 2016; root:xnu-3248.40.184~3release_x86_64 x86_64 '
    config_args='-Dprefix=/trw/local/perl/5.24.0-RC1 -Dcf_email=wyant@cpan.org -d'
    hint=recommended, useposix=true, d_sigaction=define
    useithreads=undef, usemultiplicity=undef
    use64bitint=define, use64bitall=define, uselongdouble=undef
    usemymalloc=n, bincompat5005=undef
  Compiler:
    cc='cc', ccflags ='-fno-common -DPERL_DARWIN -mmacosx-version-min=10.11 -fno-strict-aliasing -pipe -fstack-protector-strong -I/opt/local/include -DPERL_USE_SAFE_PUTENV',
    optimize='-O3',
    cppflags='-fno-common -DPERL_DARWIN -mmacosx-version-min=10.11 -fno-strict-aliasing -pipe -fstack-protector-strong -I/opt/local/include'
    ccversion='', gccversion='4.2.1 Compatible Apple LLVM 7.3.0 (clang-703.0.29)', gccosandvers=''
    intsize=4, longsize=8, ptrsize=8, doublesize=8, byteorder=12345678, doublekind=3
    d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16, longdblkind=3
    ivtype='long', ivsize=8, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8
    alignbytes=8, prototype=define
  Linker and Libraries:
    ld='cc', ldflags =' -mmacosx-version-min=10.11 -fstack-protector-strong -L/opt/local/lib'
    libpth=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../lib/clang/7.3.0/lib /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib /usr/lib /opt/local/lib
    libs=-lpthread -lgdbm -ldbm -ldl -lm -lutil -lc
    perllibs=-lpthread -ldl -lm -lutil -lc
    libc=, so=dylib, useshrplib=false, libperl=libperl.a
    gnulibc_version=''
  Dynamic Linking:
    dlsrc=dl_dlopen.xs, dlext=bundle, d_dlsymun=undef, ccdlflags=' '
    cccdlflags=' ', lddlflags=' -mmacosx-version-min=10.11 -bundle -undefined dynamic_lookup -L/opt/local/lib -fstack-protector-strong'

Locally applied patches:
    RC1


@INC for perl 5.24.0:
    /trw/local/perl/5.24.0-RC1/lib/site_perl/5.24.0/darwin-2level
    /trw/local/perl/5.24.0-RC1/lib/site_perl/5.24.0
    /trw/local/perl/5.24.0-RC1/lib/5.24.0/darwin-2level
    /trw/local/perl/5.24.0-RC1/lib/5.24.0
    .


Environment for perl 5.24.0:
    DYLD_LIBRARY_PATH (unset)
    HOME=/Users/tom
    LANG=en_US.UTF-8
    LANGUAGE (unset)
    LD_LIBRARY_PATH (unset)
    LOGDIR (unset)
    PATH=/trw/local/perl/5.24.0-RC1/bin:/trw/local/lynx/2.8.8-with-ssl/bin:/trw/local/perl6/2015.12/bin:/trw/local/perl6/2015.12/share/perl6/site/bin:/trw/local/simh/3.9-0/bin:/opt/local/bin:/opt/local/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Users/tom/Code/Tools
    PERLPKGOPT=-noppm
    PERL_BADLANG (unset)
    SHELL=/bin/bash

@p5pRT
Copy link
Author

p5pRT commented Apr 14, 2016

From @khwilliamson

On 04/14/2016 08​:59 AM, Tom Wyant (via RT) wrote​:

# New Ticket Created by Tom Wyant
# Please include the string​: [perl #127899]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl5/Ticket/Display.html?id=127899 >

This is a bug report for perl from wyant@​cpan.org,
generated with the help of perlbug 1.40 running under perl 5.24.0.

-----------------------------------------------------------------
[Please describe your issue here]
Perldelta for 5.24.1-RC1 and 5.22.2-RC1 contain, under "Selected Bug
Fixes", the item

 \*   Certain regex patterns involving a complemented POSIX class in an
     inverted bracketed character class\, and matching something else
     optionally would improperly fail to match\. An example of one that
     could fail is "/qr/\_?\[^\\Wbar\]\\x\{100\}/"\. This has been fixed\. \[perl
     \#127537\]

It looks to me like the leading slash in the example given on the last
full line is spurious. Or maybe the first three characters "/qr". At any
rate, the intent of the author is unclear, at least to this reader.

Thank you very much for Perl 5.24.0 and 5.22.2.

Thanks for reporting this.

This has now been fixed in 5.24 blead with commit
de1d2c7, with rjbs' approval. (We
already know another RC is coming)

Steve Hay will have to decide about 5.22.2.

@p5pRT
Copy link
Author

p5pRT commented Apr 14, 2016

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Apr 15, 2016

From @steve-m-hay

On Thu Apr 14 10​:01​:19 2016, public@​khwilliamson.com wrote​:

On 04/14/2016 08​:59 AM, Tom Wyant (via RT) wrote​:

# New Ticket Created by Tom Wyant
# Please include the string​: [perl #127899]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl5/Ticket/Display.html?id=127899 >

This is a bug report for perl from wyant@​cpan.org,
generated with the help of perlbug 1.40 running under perl 5.24.0.

-----------------------------------------------------------------
[Please describe your issue here]
Perldelta for 5.24.1-RC1 and 5.22.2-RC1 contain, under "Selected Bug
Fixes", the item

* Certain regex patterns involving a complemented POSIX class in an
inverted bracketed character class, and matching something else
optionally would improperly fail to match. An example of one that
could fail is "/qr/_?[^\Wbar]\x{100}/". This has been fixed.
[perl
#127537]

It looks to me like the leading slash in the example given on the
last
full line is spurious. Or maybe the first three characters "/qr". At
any
rate, the intent of the author is unclear, at least to this reader.

Thank you very much for Perl 5.24.0 and 5.22.2.

Thanks for reporting this.

This has now been fixed in 5.24 blead with commit
de1d2c7, with rjbs' approval. (We
already know another RC is coming)

Steve Hay will have to decide about 5.22.2.

Thanks, now fixed for 5.22.2 in commit 3e2691e.

@p5pRT
Copy link
Author

p5pRT commented Apr 15, 2016

@steve-m-hay - Status changed from 'open' to 'pending release'

@p5pRT
Copy link
Author

p5pRT commented May 13, 2016

From @khwilliamson

Thank you for submitting this report. You have helped make Perl better.
 
With the release of Perl 5.24.0 on May 9, 2016, this and 149 other issues have been resolved.

Perl 5.24.0 may be downloaded via https://metacpan.org/release/RJBS/perl-5.24.0

@p5pRT
Copy link
Author

p5pRT commented May 13, 2016

@khwilliamson - Status changed from 'pending release' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant