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

.trans after .subst blows up in Rakudo #1429

Closed
p6rt opened this issue Dec 7, 2009 · 7 comments
Closed

.trans after .subst blows up in Rakudo #1429

p6rt opened this issue Dec 7, 2009 · 7 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Dec 7, 2009

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

Searchable as RT71088$

@p6rt
Copy link
Author

p6rt commented Dec 7, 2009

From @masak

<masak> rakudo​: "".subst(/x/, "").trans()
<p6eval> rakudo 7ef386​: too few positional arguments​: 1 passed, 2 (or
more) expected [...]
<masak> is this one known?
<masak> it looks familiar.
<mathw> Hmm
<mathw> It does
<mathw> But that might be because we've had similar bugs in the past
<masak> it's the getting-Parrot-strings bug, I know.
<masak> but is this particular instance reported?
<mathw> I don't know
* masak submits rakudobug just to be safe
<mathw> :)

@p6rt
Copy link
Author

p6rt commented Jun 29, 2010

From @bbkr

On Kiev build​:

[16​:44] <bbkr> rakudo​: "".subst(/x/, "").trans()
[16​:45] <p6eval> rakudo 1576d4​: ( no output )

taking ticket for tests

@p6rt
Copy link
Author

p6rt commented Jun 29, 2010

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

@p6rt
Copy link
Author

p6rt commented Jul 28, 2010

From @felliott

Hello,

I believe this is fixed as of commit
4bf6c0f7bd4fe3be4df2f2f11a8d09147a172ad6

Cheers,
Fitz Elliott

@p6rt
Copy link
Author

p6rt commented Jul 30, 2010

From @bbkr

rechecked on * release
tests in t/spec/S05-transliteration/trans.t

@p6rt
Copy link
Author

p6rt commented Jul 30, 2010

@bbkr - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed Jul 30, 2010
@p6rt
Copy link
Author

p6rt commented Jul 30, 2010

From @kyleha

This is an automatically generated mail to inform you that tests are now available in t/spec/S05-transliteration/trans.t

commit c960d472288a7e5a97408a102db3ac9e4e823439
Author​: bbkr <bbkr@​c213334d-75ef-0310-aa23-eaa082d1ae64>
Date​: Fri Jul 30 17​:28​:00 2010 +0000

  [t/spec] tests for RT #​71088 .trans after .subst blows up in Rakudo
 
  git-svn-id​: http://svn.pugscode.org/pugs@&#8203;31874 c213334d-75ef-0310-aa23-eaa082d1ae64

Inline Patch
diff --git a/t/spec/S05-transliteration/trans.t b/t/spec/S05-transliteration/trans.t
index 2f96de4..816f3fb 100644
--- a/t/spec/S05-transliteration/trans.t
+++ b/t/spec/S05-transliteration/trans.t
@@ -10,7 +10,7 @@ String transliteration
 
 # L<S05/Transliteration>
 
-plan 52;
+plan 53;
 
 is("ABC".trans( ('A'=>'a'), ('B'=>'b'), ('C'=>'c') ),
     "abc",
@@ -235,4 +235,10 @@ is($_, "X  \t", 'tr/// on $_ with explicit character lists');
 # y/// is dead
 eval_dies_ok('$_ = "axbycz"; y/abc/def/', 'y/// does not exist any longer');
 
+# RT #71088
+{
+    lives_ok { "".subst(/x/, "").trans() },
+        'trans on subst output lives';
+}
+
 # vim: ft=perl6

@p6rt p6rt added the Bug label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant