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

Rakudo doesn't allow you to call a subroutine if you chose to call it 's' #1274

Closed
p6rt opened this issue Sep 7, 2009 · 5 comments
Closed
Labels

Comments

@p6rt
Copy link

p6rt commented Sep 7, 2009

Migrated from rt.perl.org#69044 (status was 'rejected')

Searchable as RT69044$

@p6rt
Copy link
Author

p6rt commented Sep 7, 2009

From @masak

<masak> rakudo​: sub s {}; s
<p6eval> rakudo 205733​: OUTPUT«s/// not implemented, try .subst as
workaround [...]
<masak> o_0
<masak> moritz_​: what's the meaning of this? :P
<TimToady> he wants you to translate to Sweutch, I guess
<moritz_> masak​: of what? bug!
* masak zubmitz ze bug

@p6rt
Copy link
Author

p6rt commented Oct 11, 2009

From @kyleha

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

commit 5acf484a284f8e60d042bced81dadcdb62cedf5e
Author​: kyle <kyle@​c213334d-75ef-0310-aa23-eaa082d1ae64>
Date​: Sun Oct 11 03​:27​:17 2009 +0000

  [t/spec] Test for RT #​69044
 
  git-svn-id​: http://svn.pugscode.org/pugs@&#8203;28745 c213334d-75ef-0310-aa23-eaa082d1ae64

Inline Patch
diff --git a/t/spec/S05-substitution/subst.t b/t/spec/S05-substitution/subst.t
index ac3b1e0..9320dd0 100644
--- a/t/spec/S05-substitution/subst.t
+++ b/t/spec/S05-substitution/subst.t
@@ -2,7 +2,7 @@ use v6;
 
 use Test;
 
-plan 76;
+plan *;
 
 # L<S05/Substitution/>
 
@@ -195,5 +195,14 @@ is '12'.subst(/(.)(.)/,{$()*2}),'24', '.. and do nifty things in closures';
     }
 }
 
+# RT #69044
+{
+    sub s { 'sub s' }
+    #?rakudo skip 'RT 69044'
+    is s,   'sub s', 'can call sub s as "s"';
+    is s(), 'sub s', 'can call sub s as "s()"';
+}
+
+done_testing;
 
 # vim: ft=perl6

@p6rt
Copy link
Author

p6rt commented Oct 11, 2009

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

@p6rt
Copy link
Author

p6rt commented Oct 28, 2009

From @kyleha

The tests have been updated (in r28939) to reflect the fact that 'bare s
is always substitution'. You can call a sub named 's', as long as you
call it as 's( ... )', so I'm marking this as rejected.

@p6rt
Copy link
Author

p6rt commented Oct 28, 2009

@kyleha - Status changed from 'open' to 'rejected'

@p6rt p6rt closed this as completed Oct 28, 2009
@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