Re: [svn:parrot] r30286 - in trunk: src/pmc t/pmc

2008-08-19 Thread Will Coleda
On Tue, Aug 19, 2008 at 1:47 PM, chromatic <[EMAIL PROTECTED]> wrote: > On Mon, Aug 18, 2008 at 04:49:59PM -0400, Will Coleda wrote: > >> > I'm pretty sure that Rakudo and Tcl want different things from NameSpaces. >> > This code has a strong smell. For those playing at home: This code is taking a

Re: [svn:parrot] r30286 - in trunk: src/pmc t/pmc

2008-08-19 Thread chromatic
On Mon, Aug 18, 2008 at 04:49:59PM -0400, Will Coleda wrote: > > I'm pretty sure that Rakudo and Tcl want different things from NameSpaces. > > This code has a strong smell. > Can you be more specific as to the odor? Do namespaces nest? Can you access them with String keys? How about Key keys?

Re: [svn:parrot] r30286 - in trunk: src/pmc t/pmc

2008-08-18 Thread Will Coleda
On Mon, Aug 18, 2008 at 4:39 PM, chromatic <[EMAIL PROTECTED]> wrote: > On Mon, Aug 18, 2008 at 12:55:35PM -0400, Will Coleda wrote: > >> This causes languages/tcl/cmd_namespace.t to fail. (Reverting this one >> change locally allows the test to pass). > > I'm pretty sure that Rakudo and Tcl want d

Re: [svn:parrot] r30286 - in trunk: src/pmc t/pmc

2008-08-18 Thread chromatic
On Mon, Aug 18, 2008 at 12:55:35PM -0400, Will Coleda wrote: > This causes languages/tcl/cmd_namespace.t to fail. (Reverting this one > change locally allows the test to pass). I'm pretty sure that Rakudo and Tcl want different things from NameSpaces. This code has a strong smell. -- Oh yeah.

Re: [svn:parrot] r30286 - in trunk: src/pmc t/pmc

2008-08-18 Thread Will Coleda
On Mon, Aug 18, 2008 at 12:55 PM, Will Coleda <[EMAIL PROTECTED]> wrote: > On Sun, Aug 17, 2008 at 5:49 PM, <[EMAIL PROTECTED]> wrote: >> Author: chromatic >> Date: Sun Aug 17 14:49:39 2008 >> New Revision: 30286 >> >> Modified: >> trunk/src/pmc/namespace.pmc >> trunk/t/pmc/namespace.t >> >> L

Re: [svn:parrot] r30286 - in trunk: src/pmc t/pmc

2008-08-18 Thread Will Coleda
On Sun, Aug 17, 2008 at 5:49 PM, <[EMAIL PROTECTED]> wrote: > Author: chromatic > Date: Sun Aug 17 14:49:39 2008 > New Revision: 30286 > > Modified: > trunk/src/pmc/namespace.pmc > trunk/t/pmc/namespace.t > > Log: > [PMC] Fixed co-recursion bug in NameSpace, when iterating through its > conte