Approved!
On Nov 27, 2007, at 5:31 PM, P T Withington wrote:
Change 20071127-ptw-C by [EMAIL PROTECTED] on 2007-11-27
16:41:07 EST
in /Users/ptw/OpenLaszlo/ringding-2
for http://svn.openlaszlo.org/openlaszlo/trunk
Summary: Correct handling of comma-separated var expressions
Bugs Fixed:
LPP-5129 ' lzo's compile differently? for (var i = 0, len =
childNodes.length; i < len; i++) { causes error'
Technical Reviewer: [EMAIL PROTECTED] (pending)
QA Reviewer: hminsky (pending)
Details:
JavascriptGenerator: Handle variableStatement, consolidate
isExpression
CommonGenerator: Handle new AST type VariableDeclarationList, mov
base isExpression here
ASTVisitor: Add interfaces for VariableDeclaration and
VariableDeclarationList
Compiler: A VariableStatement cannot be handled as a Statement.
Implement correct unparsing of VariableStatement,
VariableDeclarationList, and VariableDeclaration.
CodeGenerator: consolidate isExpression
Parser: Parse comma-separated variable declarations into new AST
VariableDeclarationList
Tests:
smokecheck
observed that test case in bug report is correctly passed through
for dhtml
Files:
M WEB-INF/lps/server/src/org/openlaszlo/sc/
JavascriptGenerator.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/CommonGenerator.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/ASTVisitor.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/Compiler.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/CodeGenerator.java
M WEB-INF/lps/server/sc/src/org/openlaszlo/sc/Parser.jjt
Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20071127-
ptw-C.tar
--
Don Anderson
Java/C/C++, Berkeley DB, systems consultant
Voice: 617-547-7881
Email: [EMAIL PROTECTED]
WWW: http://www.ddanderson.com