Some fixes to split methods

2008-09-21 Thread Chris Davaz
I have implemented the limit parameter on both Str.split(String, Integer) and Str.split(Regex, Integer). In doing so I had to change the method signature of Str.split(String) to ".sub 'split' :method :multi(_, _)" from ".sub 'split' :method :multi('String')". The former method signature is the corr

Re: method signature issues

2008-09-21 Thread Chris Davaz
Awesome Patrick, you totally nailed it ;-) I'll be submitting a patch soon. Do you know if there is a Parrot bug logged for the problem you described? On Mon, Sep 22, 2008 at 12:27 AM, Patrick R. Michaud <[EMAIL PROTECTED]> wrote: > On Sat, Sep 20, 2008 at 12:46:53PM +0800, Chris Davaz wrote: >>

Subroutine parameter with trait and default.

2008-09-21 Thread Michael G Schwern
I'm pondering what the proper syntax is for a subroutine parameter with both a trait and a default. That is... sub foo ($arg = 42) and sub foo ($arg is readonly) together in one parameter. Would that be sub foo ($arg = 42 is readonly) or sub foo ($arg is rea

Are keys, values, kv, pairs, etc. named unaries?

2008-09-21 Thread Patrick R. Michaud
According to the test suite (t/spec/S29-array/kv.t), the .kv method is defined on arrays to produce an interleaved list of indices and values: my @array = ; say @array.kv.perl; # [ 0, "a", 1, "b", 2, "c", 3, "d" ] The kv.t file also shows a functional form of kv(): my @array = ;

Re: [perl #58924] [BUG] my %a; %a = 't'; for keys %a { .say } misbehaving

2008-09-21 Thread Patrick R. Michaud
On Thu, Sep 18, 2008 at 04:12:26AM -0700, [EMAIL PROTECTED] via RT wrote: > On Tue Sep 16 08:56:29 2008, cognominal wrote: > > It prints "0\n" instead of the expected "i\n" > > > Fixed this in 31227, and values, and also cleaned up kv. They are now > multis that we use !EXPORT on (kv had been writ

Re: [svn:parrot] r31324 - trunk/src/pmc

2008-09-21 Thread NotFound
>> +=item C >> + >> +Removes the element at C. >> + >> +=cut >> + >> +*/ >> + >> +VTABLE void delete_keyed_int(INTVAL key) { >> +INTVAL size = PMC_int_val(SELF); > > With the use of the set_integer_native entry later, this should probably be: > >INTVAL size = SELF.get_integer();

Re: New Parrot mailing list

2008-09-21 Thread Moritz Lenz
Allison Randal wrote: > The new Parrot mailing list (replacing perl6-internals/parrot-porters) > is <[EMAIL PROTECTED]>. If you were subscribed to the old > list, you're now subscribed to the new list. That doesn't seem to have worked out quite right. I sent a mail today to [EMAIL PROTECTED], an

Re: [svn:parrot] r31324 - trunk/src/pmc

2008-09-21 Thread chromatic
On Sunday 21 September 2008 14:00:59 [EMAIL PROTECTED] wrote: > Log: > add delete_keyed_int to resizableintegerarray > > --- trunk/src/pmc/resizableintegerarray.pmc (original) > +++ trunk/src/pmc/resizableintegerarray.pmc Sun Sep 21 14:00:58 2008 > @@ -206,6 +206,24 @@ >          mem_sys_m

Re: method signature issues

2008-09-21 Thread Jonathan Scott Duff
How does Parrot tell which parameters are the invocants? PDD27 mentions that parameters can be decorated with :invocant, but that doesn't seem to be used anywhere. I'll also note that if :multi and :method don't go together then some of the language in PDD27 needs to change. For instance, "The C<

[perl #59068] Re: [november] Re: .perl fix

2008-09-21 Thread Moritz Lenz
Patrick R. Michaud wrote: > On Sun, Sep 21, 2008 at 03:34:50PM +0200, Carl Mäsak wrote: >> >> Andrew (>): >> > It turns out that .perl method does not escape >> > dollars >> >> Have you thought about reporting this via [EMAIL PROTECTED] > > It's already there as RT #59068. > > It would help if

pugs r22304 - [Book] A few small updates to chapters 6 and 7

2008-09-21 Thread Moritz Lenz
Index: docs/tutorial/ch06_objects.pod === --- docs/tutorial/ch06_objects.pod (revision 22303) +++ docs/tutorial/ch06_objects.pod (revision 22304) @@ -48,11 +47,25 @@ ... } +And an embedded class: + + class Heart::

Re: [svn:parrot] r31305 - in branches/pdd27mmd: include/parrot src

2008-09-21 Thread chromatic
On Sunday 21 September 2008 03:17:18 [EMAIL PROTECTED] wrote: > --- branches/pdd27mmd/src/multidispatch.c   (original) > +++ branches/pdd27mmd/src/multidispatch.c   Sun Sep 21 03:17:16 2008 > @@ -599,6 +592,7 @@ >  #endif >   >      Parrot_pcc_invoke_sub_from_sig_object(interp, sub, sig_ob

Re: method signature issues

2008-09-21 Thread Patrick R. Michaud
On Sat, Sep 20, 2008 at 12:46:53PM +0800, Chris Davaz wrote: > In any-str.pir we need to figure out how to change > .sub 'split' :method :multi('String') > into > .sub 'split' :method :multi(_, 'String') > [...] ... let's back up a bit and look at what is really happening. > [...] Any when we cha

Re: method signature issues

2008-09-21 Thread chromatic
On Sunday 21 September 2008 03:53:31 Jonathan Worthington wrote: > > I said they're incompatible -- meant in terms of their semantics.  I > > didn't say they don't work together in some cases with our current > > implementation. (They probably shouldn't.) > They probably should. If they didn't, t

Re: method signature issues

2008-09-21 Thread Chris Davaz
Patrick, Any thoughts on why I am getting the "No applicable methods" error as described in the head of this thread? On Sun, Sep 21, 2008 at 11:38 PM, Patrick R. Michaud <[EMAIL PROTECTED]> wrote: > On Sat, Sep 20, 2008 at 11:05:34PM -0700, chromatic wrote: >> On Saturday 20 September 2008 22:24:

Re: [perl #59104] export sub from used module is broken if make pir

2008-09-21 Thread Patrick R. Michaud
On Fri, Sep 19, 2008 at 10:45:30PM -0700, Ilya Belikin wrote: > # New Ticket Created by Ilya Belikin > # Please include the string: [perl #59104] > # in the subject line of all future correspondence about this issue. > # http://rt.perl.org/rt3/Ticket/Display.html?id=59104 > > > > Hi, > I comp

Re: method signature issues

2008-09-21 Thread Patrick R. Michaud
On Sat, Sep 20, 2008 at 11:05:34PM -0700, chromatic wrote: > On Saturday 20 September 2008 22:24:52 Chris Davaz wrote: > > > If it is the case that :method and :multi are incompatible, I am a bit > > surprised to see that in the Rakudo src directory: > > I said they're incompatible -- meant in te

[perl #59104] export sub from used module is broken if make pir

2008-09-21 Thread via RT
# New Ticket Created by Ilya Belikin # Please include the string: [perl #59104] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=59104 > Hi, I compile parrot r31282 and November do not export sub from modules on pir (in r3126

[svn:parrot-pdd] r31294 - in trunk: compilers/bcg/t compilers/pct/src/PAST compilers/pct/src/PCT docs/pdds languages/APL/src/parser languages/WMLScript languages/WMLScript/src languages/dotnet/build l

2008-09-21 Thread tene
Author: tene Date: Sun Sep 21 01:47:14 2008 New Revision: 31294 Modified: trunk/docs/pdds/pdd19_pir.pod Changes in other areas also in this revision: Modified: trunk/compilers/bcg/t/BCG.t trunk/compilers/pct/src/PAST/Compiler.pir trunk/compilers/pct/src/PCT/HLLCompiler.pir trunk/la

[perl #59120] Class attributes on roles do not work

2008-09-21 Thread Carl Mäsak
# New Ticket Created by "Carl Mäsak" # Please include the string: [perl #59120] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=59120 > Rakudo r31286 does not like class attributes within roles. $ ./perl6 -e 'role A { my $.

[perl #59112] Failing test in t/examples/library.t

2008-09-21 Thread Carl Mäsak
# New Ticket Created by "Carl Mäsak" # Please include the string: [perl #59112] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=59112 > In Parrot r31286: $ perl t/examples/library.t 1..4 ok 1 - examples/library/getopt_demo.

[perl #59118] Class attributes without accessors don't work in Rakudo

2008-09-21 Thread Carl Mäsak
# New Ticket Created by "Carl Mäsak" # Please include the string: [perl #59118] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=59118 > Rakudo r31286 doesn't seem to recognize class attributes with the '!' twigil. $ ./perl6

[perl #58796] [CAGE] src/library.c (and others?) refer to .past files, which are no longer used

2008-09-21 Thread Christoph Otto via RT
On Fri Sep 12 10:06:01 2008, [EMAIL PROTECTED] wrote: > src/library.c (and others?) refer to .past files, which are no longer > used. remove all references to this filetype from the parrot repo. > ~jerry This one should be closeable as of r31284. I tried acking through all instances of the strin

Re: Should $.foo attributes without "is rw" be writable from within the class

2008-09-21 Thread Daniel Ruoso
Sex, 2008-09-19 às 10:25 -0700, Jon Lang escreveu: > Daniel Ruoso wrote: > > In SMOP, it is handled based on the package of the Class, the private > > storage inside the object is something like > > $obj.^!private_storage<$!bar> > > and > > $ojb.^!private_storage<$!bar> > Note that this ought o

[perl #57776] [BUG] PIO_buf_read segfault

2008-09-21 Thread Stephane Payrard via RT
A better patch that stats the opened file. That avoids possible race conditions and is faster. thx to jonathan and NotFound for pointing the shortcomings of the previous patch. # now the test pass ok 3 - slurp() on directories fails --- ./src/io/io_unix.c.orig 2008-09-19 17:56:19.0

[perl #59082] call bug in the loop

2008-09-21 Thread via RT
# New Ticket Created by Ilya Belikin # Please include the string: [perl #59082] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=59082 > Hello, we catch bug: my $max = 13; cl() for 1..2; # print only \n! cl(); # print 13 s

[perl #57776] [BUG] PIO_buf_read segfault

2008-09-21 Thread Stephane Payrard via RT
the problematic test is: slurp("./t") Unix accepts to open a directory but that leads to problem down the pipe. This following patch, apparently innocent, should fix the problem by refusing to open a directory but it triggers a memory bug. ../../parrot ../../runtime/parrot/library/PGE/Per

Re: method signature issues

2008-09-21 Thread Jonathan Worthington
chromatic wrote: On Saturday 20 September 2008 22:24:52 Chris Davaz wrote: If it is the case that :method and :multi are incompatible, I am a bit surprised to see that in the Rakudo src directory: I said they're incompatible -- meant in terms of their semantics. I didn't say they do

Re: throw oddities in pdd23

2008-09-21 Thread Stephen Weeks
Not long ago, Stephen Weeks proclaimed... > Not long ago, Allison Randal proclaimed... > > Apologies if my comments on this thread and update to the exceptions PDD > > weren't clear. The resume continuation should continue to live within > > the exception object, not be passed as a separate argu