Change 34129 by [EMAIL PROTECTED] on 2008/07/11 20:04:57
Regression tests for, and fix quite a lot of bugs in, cases FF_LINESNGL
and FF_LINEGLOB in pp_formline. (While investigating RT #55668)
Sometimes I wonder if this is actually fun.
Affected files ...
... //depot/perl/pp_ctl.c#696 edit
... //depot/perl/t/op/write.t#50 edit
Differences ...
==== //depot/perl/pp_ctl.c#696 (text) ====
Index: perl/pp_ctl.c
--- perl/pp_ctl.c#695~34125~ 2008-07-11 04:20:04.000000000 -0700
+++ perl/pp_ctl.c 2008-07-11 13:04:57.000000000 -0700
@@ -503,6 +503,7 @@
*t = '\0';
sv_catpvn_utf8_upgrade(PL_formtarget, f, arg, nsv);
t = SvEND(PL_formtarget);
+ f += arg;
break;
}
if (!targ_is_utf8 && DO_UTF8(tmpForm)) {
@@ -767,9 +768,8 @@
{
const bool oneline = fpc[-1] == FF_LINESNGL;
const char *s = item = SvPV_const(sv, len);
+ item_is_utf8 = DO_UTF8(sv);
itemsize = len;
- if ((item_is_utf8 = DO_UTF8(sv)))
- itemsize = sv_len_utf8(sv);
if (itemsize) {
bool chopped = FALSE;
const char *const send = s + len;
@@ -791,8 +791,6 @@
}
}
SvCUR_set(PL_formtarget, t - SvPVX_const(PL_formtarget));
- if (targ_is_utf8)
- SvUTF8_on(PL_formtarget);
if (oneline) {
SvCUR_set(sv, chophere - item);
sv_catsv(PL_formtarget, sv);
@@ -803,8 +801,10 @@
SvCUR_set(PL_formtarget, SvCUR(PL_formtarget) - 1);
SvGROW(PL_formtarget, SvCUR(PL_formtarget) + fudge + 1);
t = SvPVX(PL_formtarget) + SvCUR(PL_formtarget);
- if (item_is_utf8)
+ if (item_is_utf8) {
targ_is_utf8 = TRUE;
+ sv_pos_b2u(sv, &itemsize);
+ }
}
break;
}
==== //depot/perl/t/op/write.t#50 (xtext) ====
Index: perl/t/op/write.t
--- perl/t/op/write.t#49~34124~ 2008-07-11 03:44:51.000000000 -0700
+++ perl/t/op/write.t 2008-07-11 13:04:57.000000000 -0700
@@ -61,7 +61,7 @@
my $bas_tests = 20;
# number of tests in section 3
-my $bug_tests = 4;
+my $bug_tests = 4 + 3 * 3 * 3;
# number of tests in section 4
my $hmb_tests = 35;
@@ -511,6 +511,27 @@
like $@, qr/Undefined format/, 'no such format';
}
+{
+ my ($pound_utf8, $pm_utf8) = map { my $a = "$_\x{100}"; chop $a; $a}
+ my ($pound, $pm) = ("\xA3", "\xB1");
+
+ foreach my $first ('N', $pound, $pound_utf8) {
+ foreach my $base ('N', $pm, $pm_utf8) {
+ foreach my $second ($base, "$base\n", "$base\nMoo!") {
+ my $name = "$first, $second";
+ $name =~ s/\n/\\n/;
+
+ my ($copy1, $copy2) = ($first, $second);
+ $first =~ /(.+)/ or die $first;
+ my $expect = "1${1}2";
+ $second =~ /(.+)/ or die $second;
+ $expect .= " 3${1}4";
+
+ is swrite('1^*2 3^*4', $copy1, $copy2), $expect, $name;
+ }
+ }
+ }
+}
format EMPTY =
.
End of Patch.