[perl #56274] [PATCH] Re: design problem with :outer

2008-07-01 Thread Patrick R. Michaud via RT
Fixed by jonathan++ and chromatic++ as of r28795. Thanks! Pm

Re: [perl #56274] [PATCH] Re: design problem with :outer

2008-06-24 Thread Patrick R. Michaud
On Tue, Jun 24, 2008 at 03:43:34PM -0700, Mehmet Yavuz Selim Soyturk via RT wrote: > There is another problem with :outer. It does not let you create inner > functions dynamically. For example, I couldn't use it to compile next > ECMAScript code: > > function outer() { > var i = 0; >

Re: [perl #56282] Re: [PATCH] Re: design problem with :outer

2008-06-24 Thread Patrick R. Michaud
Based on discussion earlier today in #parrotsketch, here's the detailed specification for resolving outer subs using :lexid . If someone wants to fold this into pdd20 somehow (on or about line 233), that would be especially helpful. Every Parrot subroutine that serves as a lexical outer

Re: [perl #56274] [PATCH] Re: design problem with :outer

2008-06-24 Thread Mehmet Yavuz Selim Soyturk
There is another problem with :outer. It does not let you create inner functions dynamically. For example, I couldn't use it to compile next ECMAScript code: function outer() { var i = 0; return eval("function() { return i++; }"); } Is it not possible to have an opcode tha

[perl #56274] [PATCH] Re: design problem with :outer

2008-06-24 Thread Patrick R. Michaud (via RT)
# New Ticket Created by Patrick R. Michaud # Please include the string: [perl #56274] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=56274 > On Mon, Jun 23, 2008 at 06:27:31PM -0500, Patrick R. Michaud wrote: > There appear

[perl #56284] Re: [PATCH] Re: design problem with :outer

2008-06-24 Thread via RT
# New Ticket Created by chromatic # Please include the string: [perl #56284] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=56284 > On Monday 23 June 2008 21:56:54 Patrick R. Michaud wrote: > Alas, the patch doesn't seem t

[perl #56282] Re: [PATCH] Re: design problem with :outer

2008-06-24 Thread Patrick R. Michaud (via RT)
# New Ticket Created by Patrick R. Michaud # Please include the string: [perl #56282] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=56282 > On Mon, Jun 23, 2008 at 09:33:43PM -0500, Patrick R. Michaud wrote: > On Mon, Jun

[perl #56278] [PATCH] Re: design problem with :outer

2008-06-24 Thread via RT
# New Ticket Created by Bob Rogers # Please include the string: [perl #56278] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=56278 > From: "Patrick R. Michaud" <[EMAIL PROTECTED]> Date: Mon, 23 Jun 2008 21:33:43 -0500

Re: [PATCH] Re: design problem with :outer

2008-06-23 Thread chromatic
On Monday 23 June 2008 21:56:54 Patrick R. Michaud wrote: > Alas, the patch doesn't seem to hold when loading things from > precompiled bytecode -- e.g., running perl6.pbc versus perl6.pir. > > So, the patch I submitted appears to be incomplete, and > now I need to figure out how :outer is resolve

Re: [PATCH] Re: design problem with :outer

2008-06-23 Thread Patrick R. Michaud
On Mon, Jun 23, 2008 at 09:33:43PM -0500, Patrick R. Michaud wrote: > On Mon, Jun 23, 2008 at 06:27:31PM -0500, Patrick R. Michaud wrote: > > There appears to be a fundamental design problem in Parrot's > > current implementation of :outer. The short summary is that > > :outer("sub_name") doesn't

[PATCH] Re: design problem with :outer

2008-06-23 Thread Patrick R. Michaud
On Mon, Jun 23, 2008 at 06:27:31PM -0500, Patrick R. Michaud wrote: > There appears to be a fundamental design problem in Parrot's > current implementation of :outer. The short summary is that > :outer("sub_name") doesn't provide sufficient specificity > to accurately resolve an outer sub. > [...]