[Laszlo-dev] For Review: Change 20071217-dda-b Summary: Support ...rest syntax for variable args in SWF9.

[Donald Anderson]

I went ahead and fixed this in the grammar, another review coming up.

On Dec 17, 2007, at 4:10 PM, P T Withington wrote:

> This is ok for now, but doesn't the Parser change imply the ... can  
> appear before any parameter (and more than once), whereas, I think  
> it is only allowed before the last parameter.  It might be better  
> to have a separate production `FormalRestParameter` that is the  
> (optional) last element of a parameter list.
>
> On 2007-12-17, at 16:02 EST, Donald Anderson wrote:
>
>> Change 20071217-dda-b by dda at freddie.local on 2007-12-17 15:58:06 EST
>>    in /Users/dda/laszlo/src/svn/openlaszlo/branches/devildog
>>    for http://svn.openlaszlo.org/openlaszlo/branches/devildog
>>
>> Summary: Support ...rest syntax for variable args in SWF9.
>>
>> New Features: varargs uses AS3 syntax.
>>
>> Bugs Fixed: none
>>
>> Technical Reviewer: ptw (pending)
>> QA Reviewer: hminsky (pending)
>> Doc Reviewer: (pending)
>>
>> Documentation: none
>>
>> Release Notes: none
>>
>> Details:
>>    Recognize the '...' syntax for formal argument lists, and pass  
>> it through to SWF9 compiler.
>>    This does not do anything for other runtimes, I'll file a JIRA  
>> and cover that separately.
>>
>> Tests:
>>    simple test to add ... to existing method and see it added on  
>> output.
>>
>> Files:
>> M      WEB-INF/lps/server/src/org/openlaszlo/sc/ 
>> SWF9ParseTreePrinter.java
>> M      WEB-INF/lps/server/sc/src/org/openlaszlo/sc/Parser.jjt
>> M      WEB-INF/lps/server/sc/src/org/openlaszlo/sc/parser/ 
>> ASTIdentifier.java
>>
>> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071217- 
>> dda-b.tar
>>
>>
>>
>> --
>>
>> Don Anderson
>> Java/C/C++, Berkeley DB, systems consultant
>>
>> Voice:  617-547-7881
>> Email:  dda at ddanderson.com
>> WWW:    http://www.ddanderson.com
>>
>


--

Don Anderson
Java/C/C++, Berkeley DB, systems consultant

Voice:  617-547-7881
Email:  dda at ddanderson.com
WWW:    http://www.ddanderson.com