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

Re-evals break Unicode char classes (Bleadperl breaks JE) #11180

Closed
p5pRT opened this issue Mar 6, 2011 · 8 comments
Closed

Re-evals break Unicode char classes (Bleadperl breaks JE) #11180

p5pRT opened this issue Mar 6, 2011 · 8 comments

Comments

@p5pRT
Copy link

p5pRT commented Mar 6, 2011

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

Searchable as RT85528$

@p5pRT
Copy link
Author

p5pRT commented Mar 6, 2011

From @cpansprout

(for about the fourth time :-)

I’m now (ab08a36) seeing this failure​:

t/15.10-regexp-objects.t ................. 1/505
# Failed test '[\ufffF]'
# at /Users/sprout/.cpan/build/JE-0.055-c49Iek/blib/lib/JE/Object/Function.pm line 460.
Wide character in print at /usr/local/lib/perl5/5.13.10/Test/Builder.pm line 1759, <DATA> line 1.
# got​: 'null'
# expected​: '�'

I don’t know whether this is related to <http​://www.cpantesters.org/cpan/report/7d721a6c-4544-11e0-ae7e-8e939dcd942a>.

I have yet to do a binary search.


Flags​:
  category=core
  severity=low


Site configuration information for perl 5.13.10​:

Configured by sprout at Mon Feb 21 22​:02​:31 PST 2011.

Summary of my perl5 (revision 5 version 13 subversion 10) configuration​:
  Snapshot of​: ab08a36
  Platform​:
  osname=darwin, osvers=10.5.0, archname=darwin-2level
  uname='darwin pint.local 10.5.0 darwin kernel version 10.5.0​: fri nov 5 23​:20​:39 pdt 2010; root​:xnu-1504.9.17~1release_i386 i386 '
  config_args='-de -Dusedevel'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=undef, usemultiplicity=undef
  useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef
  use64bitint=undef, use64bitall=undef, uselongdouble=undef
  usemymalloc=n, bincompat5005=undef
  Compiler​:
  cc='cc', ccflags ='-fno-common -DPERL_DARWIN -no-cpp-precomp -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include',
  optimize='-O3',
  cppflags='-no-cpp-precomp -fno-common -DPERL_DARWIN -no-cpp-precomp -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include'
  ccversion='', gccversion='4.2.1 (Apple Inc. build 5664)', gccosandvers=''
  intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234
  d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16
  ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8
  alignbytes=8, prototype=define
  Linker and Libraries​:
  ld='env MACOSX_DEPLOYMENT_TARGET=10.3 cc', ldflags =' -fstack-protector -L/usr/local/lib'
  libpth=/usr/local/lib /usr/lib
  libs=-ldbm -ldl -lm -lutil -lc
  perllibs=-ldl -lm -lutil -lc
  libc=/usr/lib/libc.dylib, so=dylib, useshrplib=false, libperl=libperl.a
  gnulibc_version=''
  Dynamic Linking​:
  dlsrc=dl_dlopen.xs, dlext=bundle, d_dlsymun=undef, ccdlflags=' '
  cccdlflags=' ', lddlflags=' -bundle -undefined dynamic_lookup -L/usr/local/lib -fstack-protector'

Locally applied patches​:
 


@​INC for perl 5.13.10​:
  /usr/local/lib/perl5/site_perl/5.13.10/darwin-2level
  /usr/local/lib/perl5/site_perl/5.13.10
  /usr/local/lib/perl5/5.13.10/darwin-2level
  /usr/local/lib/perl5/5.13.10
  /usr/local/lib/perl5/site_perl
  .


Environment for perl 5.13.10​:
  DYLD_LIBRARY_PATH (unset)
  HOME=/Users/sprout
  LANG=en_US.UTF-8
  LANGUAGE (unset)
  LD_LIBRARY_PATH (unset)
  LOGDIR (unset)
  PATH=/usr/bin​:/bin​:/usr/sbin​:/sbin​:/usr/local/bin​:/usr/X11/bin​:/usr/local/bin
  PERL_BADLANG (unset)
  SHELL=/bin/bash

@p5pRT
Copy link
Author

p5pRT commented Mar 6, 2011

From @cpansprout

I’ve reduced it to this​:

$ ./miniperl -Ilib -le 'print "\x{100}" =~ /(?{})[\x{100}]/'

$ ./miniperl -Ilib -le 'print "\x{100}" =~ /[\x{100}]/'
1

@p5pRT
Copy link
Author

p5pRT commented Mar 6, 2011

@cpansprout - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Mar 6, 2011

From @cpansprout

On Sun Mar 06 13​:43​:44 2011, sprout wrote​:

I’ve reduced it to this​:

$ ./miniperl -Ilib -le 'print "\x{100}" =~ /(?{})[\x{100}]/'

$ ./miniperl -Ilib -le 'print "\x{100}" =~ /[\x{100}]/'
1

This was caused by​:

commit 137165a
Author​: Karl Williamson <public@​khwilliamson.com>
Date​: Fri Feb 25 20​:10​:47 2011 -0700

  Free up bit in ANYOF flags
 
  This is the foundation for fixing the regression RT #82610. My analysis
  was wrong that two bits could be shared, at least not without further
  work. This changes to use a different mechanism to pass needed
  information to regexec.c so that another bit can be freed up and, in a
  later commit, the two bits can become unshared again.
 
  The bit that is freed up is ANYOF_UTF8, which basically said there is
  something that is matched outside the ANYOF bitmap, and requires the
  target string to be in utf8. This changes things so the existence of
  something besides the bitmap indicates this, and so no flag is needed.
  The flag bit ANYOF_NONBITMAP_NON_UTF8 remains to indicate that there is
  something that should be matched outside the bitmap even if the target
  string isn't in utf8.

@p5pRT
Copy link
Author

p5pRT commented Mar 7, 2011

From @cpansprout

On Sun Mar 06 13​:31​:43 2011, sprout wrote​:

I don’t know whether this is related to
<http​://www.cpantesters.org/cpan/report/7d721a6c-4544-11e0-ae7e-
8e939dcd942a>.

It’s not.

@p5pRT
Copy link
Author

p5pRT commented Mar 7, 2011

From [Unknown Contact. See original ticket]

On Sun Mar 06 13​:31​:43 2011, sprout wrote​:

I don’t know whether this is related to
<http​://www.cpantesters.org/cpan/report/7d721a6c-4544-11e0-ae7e-
8e939dcd942a>.

It’s not.

@p5pRT
Copy link
Author

p5pRT commented Mar 9, 2011

From @khwilliamson

Commit 137165a should have fixed this
--Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Mar 9, 2011

@khwilliamson - Status changed from 'open' 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