|
 |
85ab1f |
From 69beb4272d324bb0724b140b5ddca517e90d89b9 Mon Sep 17 00:00:00 2001
|
|
|
85ab1f |
From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <ppisar@redhat.com>
|
|
|
85ab1f |
Date: Tue, 5 Dec 2017 10:59:42 +0100
|
|
|
85ab1f |
Subject: [PATCH] in Data-Dumper, quote glob names better
|
|
|
85ab1f |
MIME-Version: 1.0
|
|
|
85ab1f |
Content-Type: text/plain; charset=UTF-8
|
|
|
85ab1f |
Content-Transfer-Encoding: 8bit
|
|
|
85ab1f |
|
|
|
85ab1f |
Ported to Data-Dumper-1.167 from perl git tree:
|
|
|
85ab1f |
|
|
|
85ab1f |
commit abda9fe0fe75ae824723761c1c98af958f17a41c
|
|
|
85ab1f |
Author: Zefram <zefram@fysh.org>
|
|
|
85ab1f |
Date: Fri Dec 1 17:35:35 2017 +0000
|
|
|
85ab1f |
|
|
|
85ab1f |
in Data-Dumper, quote glob names better
|
|
|
85ab1f |
|
|
|
85ab1f |
Glob name quoting should obey Useqq. Fixes [perl #119831].
|
|
|
85ab1f |
|
|
|
85ab1f |
Signed-off-by: Petr Písař <ppisar@redhat.com>
|
|
|
85ab1f |
---
|
|
|
85ab1f |
Dumper.pm | 4 ++--
|
|
|
85ab1f |
Dumper.xs | 22 +++++++---------------
|
|
|
85ab1f |
t/dumper.t | 35 ++++++++++++++++++++++++++++++++++-
|
|
|
85ab1f |
3 files changed, 43 insertions(+), 18 deletions(-)
|
|
|
85ab1f |
|
|
|
85ab1f |
diff --git a/Dumper.pm b/Dumper.pm
|
|
|
85ab1f |
index 00f6326..696964a 100644
|
|
|
85ab1f |
--- a/Dumper.pm
|
|
|
85ab1f |
+++ b/Dumper.pm
|
|
|
85ab1f |
@@ -527,8 +527,8 @@ sub _dump {
|
|
|
85ab1f |
$ref = \$val;
|
|
|
85ab1f |
if (ref($ref) eq 'GLOB') { # glob
|
|
|
85ab1f |
my $name = substr($val, 1);
|
|
|
85ab1f |
- if ($name =~ /^[A-Za-z_][\w:]*$/ && $name ne 'main::') {
|
|
|
85ab1f |
- $name =~ s/^main::/::/;
|
|
|
85ab1f |
+ $name =~ s/^main::(?!\z)/::/;
|
|
|
85ab1f |
+ if ($name =~ /\A(?:[A-Z_a-z][0-9A-Z_a-z]*)?::(?:[0-9A-Z_a-z]+::)*[0-9A-Z_a-z]*\z/ && $name ne 'main::') {
|
|
|
85ab1f |
$sname = $name;
|
|
|
85ab1f |
}
|
|
|
85ab1f |
else {
|
|
|
85ab1f |
diff --git a/Dumper.xs b/Dumper.xs
|
|
|
85ab1f |
index 5a21721..8a16e04 100644
|
|
|
85ab1f |
--- a/Dumper.xs
|
|
|
85ab1f |
+++ b/Dumper.xs
|
|
|
85ab1f |
@@ -1300,29 +1300,21 @@ DD_dump(pTHX_ SV *val, const char *name, STRLEN namelen, SV *retval, HV *seenhv,
|
|
|
85ab1f |
i = 0; else i -= 4;
|
|
|
85ab1f |
}
|
|
|
85ab1f |
if (globname_needs_quote(c,i)) {
|
|
|
85ab1f |
-#ifdef GvNAMEUTF8
|
|
|
85ab1f |
- if (GvNAMEUTF8(val)) {
|
|
|
85ab1f |
sv_grow(retval, SvCUR(retval)+2);
|
|
|
85ab1f |
r = SvPVX(retval)+SvCUR(retval);
|
|
|
85ab1f |
r[0] = '*'; r[1] = '{';
|
|
|
85ab1f |
SvCUR_set(retval, SvCUR(retval)+2);
|
|
|
85ab1f |
- esc_q_utf8(aTHX_ retval, c, i, 1, style->useqq);
|
|
|
85ab1f |
+ esc_q_utf8(aTHX_ retval, c, i,
|
|
|
85ab1f |
+#ifdef GvNAMEUTF8
|
|
|
85ab1f |
+ !!GvNAMEUTF8(val)
|
|
|
85ab1f |
+#else
|
|
|
85ab1f |
+ 0
|
|
|
85ab1f |
+#endif
|
|
|
85ab1f |
+ , style->useqq);
|
|
|
85ab1f |
sv_grow(retval, SvCUR(retval)+2);
|
|
|
85ab1f |
r = SvPVX(retval)+SvCUR(retval);
|
|
|
85ab1f |
r[0] = '}'; r[1] = '\0';
|
|
|
85ab1f |
i = 1;
|
|
|
85ab1f |
- }
|
|
|
85ab1f |
- else
|
|
|
85ab1f |
-#endif
|
|
|
85ab1f |
- {
|
|
|
85ab1f |
- sv_grow(retval, SvCUR(retval)+6+2*i);
|
|
|
85ab1f |
- r = SvPVX(retval)+SvCUR(retval);
|
|
|
85ab1f |
- r[0] = '*'; r[1] = '{'; r[2] = '\'';
|
|
|
85ab1f |
- i += esc_q(r+3, c, i);
|
|
|
85ab1f |
- i += 3;
|
|
|
85ab1f |
- r[i++] = '\''; r[i++] = '}';
|
|
|
85ab1f |
- r[i] = '\0';
|
|
|
85ab1f |
- }
|
|
|
85ab1f |
}
|
|
|
85ab1f |
else {
|
|
|
85ab1f |
sv_grow(retval, SvCUR(retval)+i+2);
|
|
|
85ab1f |
diff --git a/t/dumper.t b/t/dumper.t
|
|
|
85ab1f |
index 643160a..0c12f34 100644
|
|
|
85ab1f |
--- a/t/dumper.t
|
|
|
85ab1f |
+++ b/t/dumper.t
|
|
|
85ab1f |
@@ -108,7 +108,7 @@ sub SKIP_TEST {
|
|
|
85ab1f |
++$TNUM; print "ok $TNUM # skip $reason\n";
|
|
|
85ab1f |
}
|
|
|
85ab1f |
|
|
|
85ab1f |
-$TMAX = 450;
|
|
|
85ab1f |
+$TMAX = 456;
|
|
|
85ab1f |
|
|
|
85ab1f |
# Force Data::Dumper::Dump to use perl. We test Dumpxs explicitly by calling
|
|
|
85ab1f |
# it direct. Out here it lets us knobble the next if to test that the perl
|
|
|
85ab1f |
@@ -1740,3 +1740,36 @@ EOT
|
|
|
85ab1f |
TEST (qq(Dumper("\n")), '\n alone');
|
|
|
85ab1f |
TEST (qq(Data::Dumper::DumperX("\n")), '\n alone') if $XS;
|
|
|
85ab1f |
}
|
|
|
85ab1f |
+#############
|
|
|
85ab1f |
+our @globs = map { $_, \$_ } map { *$_ } map { $_, "s::$_" }
|
|
|
85ab1f |
+ "foo", "\1bar", "L\x{e9}on", "m\x{100}cron", "snow\x{2603}";
|
|
|
85ab1f |
+$WANT = <<'EOT';
|
|
|
85ab1f |
+#$globs = [
|
|
|
85ab1f |
+# *::foo,
|
|
|
85ab1f |
+# \*::foo,
|
|
|
85ab1f |
+# *s::foo,
|
|
|
85ab1f |
+# \*s::foo,
|
|
|
85ab1f |
+# *{"::\1bar"},
|
|
|
85ab1f |
+# \*{"::\1bar"},
|
|
|
85ab1f |
+# *{"s::\1bar"},
|
|
|
85ab1f |
+# \*{"s::\1bar"},
|
|
|
85ab1f |
+# *{"::L\351on"},
|
|
|
85ab1f |
+# \*{"::L\351on"},
|
|
|
85ab1f |
+# *{"s::L\351on"},
|
|
|
85ab1f |
+# \*{"s::L\351on"},
|
|
|
85ab1f |
+# *{"::m\x{100}cron"},
|
|
|
85ab1f |
+# \*{"::m\x{100}cron"},
|
|
|
85ab1f |
+# *{"s::m\x{100}cron"},
|
|
|
85ab1f |
+# \*{"s::m\x{100}cron"},
|
|
|
85ab1f |
+# *{"::snow\x{2603}"},
|
|
|
85ab1f |
+# \*{"::snow\x{2603}"},
|
|
|
85ab1f |
+# *{"s::snow\x{2603}"},
|
|
|
85ab1f |
+# \*{"s::snow\x{2603}"}
|
|
|
85ab1f |
+#];
|
|
|
85ab1f |
+EOT
|
|
|
85ab1f |
+{
|
|
|
85ab1f |
+ local $Data::Dumper::Useqq = 1;
|
|
|
85ab1f |
+ TEST (q(Data::Dumper->Dump([\@globs], ["globs"])), 'globs: Dump()');
|
|
|
85ab1f |
+ TEST (q(Data::Dumper->Dumpxs([\@globs], ["globs"])), 'globs: Dumpxs()')
|
|
|
85ab1f |
+ if $XS;
|
|
|
85ab1f |
+}
|
|
|
85ab1f |
--
|
|
|
85ab1f |
2.13.6
|
|
|
85ab1f |
|