On Jan 3, 2008 4:58 PM, via RT Zev Benjamin
<[EMAIL PROTECTED]> wrote:
> # New Ticket Created by Zev Benjamin
> # Please include the string: [perl #49354]
> # in the subject line of all future correspondence about this issue.
> # <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=49354 >
>
>
> Attached are some tests from the NQP TODO file. I attempted to make
> reasonable guesses about where NQP diverges from perl6, but erred on the
> side of additional functionality.
>
>
thanks! i've modified your patch somewhat, as there are some test
cases that nqp won't support. most of the modifications involve the
fact that nqp has no runtime, so things like $(1) won't work, because
nqp translates $(...) to something like
$P0 = new 'Integer'
$P0 = 1
$P0.'get_scalar'()
Parrot's Integer PMC doesn't have a 'get_scalar' method, so this
fails. this means i wasn't able to apply the scalar context patch.
we'll have to come up with another way to test that, along with list
context and match variable access.
other pieces won't be supported, like :
sub f5 ($x, :short($longname)) { $x - $longname; }
as we're only allowing named parameters to have one name. if we need
that syntax, we can add it later, but have no plans for it until it's
needed.
nice work, though, this increases our test coverage and makes me more
confident that the new parameter/argument code is working well.
applied as r24538.
~jerry
