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

CORE::GLOBAL::glob doesn’t *really* override glob. #11489

Open
p5pRT opened this issue Jul 11, 2011 · 4 comments
Open

CORE::GLOBAL::glob doesn’t *really* override glob. #11489

p5pRT opened this issue Jul 11, 2011 · 4 comments

Comments

@p5pRT
Copy link

p5pRT commented Jul 11, 2011

Migrated from rt.perl.org#94480 (status was 'open')

Searchable as RT94480$

@p5pRT
Copy link
Author

p5pRT commented Jul 11, 2011

From @cpansprout

This script prints �overloaded� instead of �custom glob�, because the
custom override doesn�t work in the presence of overloading​:

use File​::Glob; # make sure it does not override ours if loaded
  # automatically
BEGIN { *CORE​::GLOBAL​::glob = sub { warn "custom glob" } }
use overload '<>' => sub { warn "overloaded" };
$o = bless[];
glob $o;

We could fix this by following the VMS path and putting the globbing code in the core. That would solve #93638, too. Or is there a better way?


Flags​:
  category=core
  severity=low


Site configuration information for perl 5.14.0​:

Configured by sprout at Wed May 11 13​:45​:58 PDT 2011.

Summary of my perl5 (revision 5 version 14 subversion 0) configuration​:
  Snapshot of​: eb70bb4
  Platform​:
  osname=darwin, osvers=10.5.0, archname=darwin-thread-multi-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='-Dusedevel -de -Duseithreads -Doptimize=-g'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=define, usemultiplicity=define
  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 -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include',
  optimize='-g',
  cppflags='-fno-common -DPERL_DARWIN -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​:
  RC3


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


Environment for perl 5.14.0​:
  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 Jul 11, 2011

From @cpansprout

Rafael Garcia-Suarez wrote​:

On 11 July 2011 03​:50, Father Chrysostomos <perlbug-followup@​perl.org>
wrote​:

This script prints �overloaded� instead of �custom glob�, because the
custom override doesnâ��t work in the presence of overloading​:

use File​::Glob; # make sure it does not override ours if loaded
               # automatically
BEGIN { *CORE​::GLOBAL​::glob = sub { warn "custom glob" } }
use overload '<>' => sub { warn "overloaded" };
$o = bless[];
glob $o;

We could fix this by following the VMS path and putting the globbing
code in the core. That would solve #93638, too. Or is there a better way?

I'm not totally sure this is a bug. glob and <> are, if I remember
correctly, exactly the same built-in (if we take <> in the glob
acceptance, not in the readline acceptance), and in that case it seems
natural that the more specific (the object overload) takes precedence
over the less specific (the global override).

But this example gives me â��custom absâ��​:

BEGIN { *CORE​::GLOBAL​::abs = sub { warn "custom abs" } }
use overload 'abs' => sub { warn "overloaded" };
$o = bless[];
abs $o;

@p5pRT
Copy link
Author

p5pRT commented Jul 11, 2011

From [Unknown Contact. See original ticket]

Rafael Garcia-Suarez wrote​:

On 11 July 2011 03​:50, Father Chrysostomos <perlbug-followup@​perl.org>
wrote​:

This script prints �overloaded� instead of �custom glob�, because the
custom override doesnâ��t work in the presence of overloading​:

use File​::Glob; # make sure it does not override ours if loaded
               # automatically
BEGIN { *CORE​::GLOBAL​::glob = sub { warn "custom glob" } }
use overload '<>' => sub { warn "overloaded" };
$o = bless[];
glob $o;

We could fix this by following the VMS path and putting the globbing
code in the core. That would solve #93638, too. Or is there a better way?

I'm not totally sure this is a bug. glob and <> are, if I remember
correctly, exactly the same built-in (if we take <> in the glob
acceptance, not in the readline acceptance), and in that case it seems
natural that the more specific (the object overload) takes precedence
over the less specific (the global override).

But this example gives me â��custom absâ��​:

BEGIN { *CORE​::GLOBAL​::abs = sub { warn "custom abs" } }
use overload 'abs' => sub { warn "overloaded" };
$o = bless[];
abs $o;

@p5pRT
Copy link
Author

p5pRT commented Jul 11, 2011

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

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

2 participants