Change 20090111-dda-y by [email protected] on 2009-01-11 10:54:15 EST
in /Users/dda/laszlo/src/svn/openlaszlo/trunk-c
for http://svn.openlaszlo.org/openlaszlo/trunk
Summary: Remove references to 'this' in static functions (revised 2)
New Features:
Bugs Fixed: LPP-7514 (Compilation error when setting
compiler.swf9.catcherrors=true in lps.properties)
Technical Reviewer: ptw (pending)
QA Reviewer: (pending)
Doc Reviewer: (pending)
Documentation:
Release Notes:
Details:
** This changeset under consideration for 4.2.0.1
[This changeset is a second revision of an earlier changeset, to
ensure the new test case
listed in the JIRA runs correctly. Everything indicated under
Tests has been retested with this revision.]
The implementation of catchFunctionExceptions (catcherrors) was
blind to static functions,
'this' cannot be used in static functions.
To tell that a function is static, this changeset peeks up the
tree to the parent
of the function node, which should be a ASTModifiedDefinition
('moddef'), and checks the static
flag there. A shortcut in copying part of the tree without the
moddef
needed to be fixed, to do that, a 'shallow copy' function was
added to the moddef class.
An alternate way of detecting static-ness is to set a 'static'
property in the options.
I had tried in addressing LPP-5813 (not yet completed), and had
some trouble getting
it to work. It's not clear which one is cleaner, but the one
that works wins for me.
This fix is slightly more encompassing than it needs to be, it
also disables any possibility
of applying the WITH_THIS directive within a static function.
This should always be
correct.
This changeset also fixes a bug in that previously,
ASTFunctionExpressions were not
considered for the 'try/catch' treatment. This effectively
excluded state methods,
and probably some other important cases. Fixing this exposed an
error in smoketest
(only shown when catcherrors is true).
Tests: 880 Failures: 1 Errors: 1
TestFailure: «<IntentionalErrors>»/testUndefinedErrors
failed: Total number of expected errors: expected 1 got 0
TestError: «<IntentionalErrors>»/testUndefinedErrors
failed: Wrong error: expected «function» got 'testing: 1, 2, 3'
ptw believes this test case should be rewritten: That error is
because that test is using arguments.callee to fake a closure (which
is not going to work because the transform moves the code to an inner
closure, and arguments.callee no longer gets it to the original
function.
Tests:
(dhtml,swf8,swf9) x (lzpix,weather,smokecheck,hello) x (enable
catcherrors, disable catcherrors)
-- smokecheck error for swf9 x smokecheck x (enable catcherrors)
as noted above.
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/CodeGenerator.java
M WEB-INF/lps/server/sc/src/org/openlaszlo/sc/parser/
ASTModifiedDefinition.java
Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090111-dda-y.tar
--
Don Anderson
Java/C/C++, Berkeley DB, systems consultant
voice: 617-306-2057
email: [email protected]
www: http://www.ddanderson.com
