Approved.

One tiny nit; you are checking del.c twice in LzTimer.


Change 20071220-maxcarlson-X by <mailto:[EMAIL PROTECTED]>[EMAIL PROTECTED] on 2007-12-20
13:13:36 PST
     in /Users/maxcarlson/openlaszlo/trunk
for <http://svn.openlaszlo.org/openlaszlo/trunk>http://svn.openlaszlo.org/openlaszlo/trunk

Summary: UPDATED - Fix enable/disabling of delegates bound to lztimer

New Features:

Bugs Fixed: LPP-5302 - LzTimer ignores disabled events

Technical Reviewer: promanik
QA Reviewer: ptw
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: Check if a context is deleted before executing the delegate.
Add more comments to keep track of inlining, fix inlining to match implementation of execute(). We need compiler inlining!!!


Tests: smoketest and testcase in LPP-5302 passes.

Files:
M      WEB-INF/lps/lfc/services/LzTimer.lzs
M      WEB-INF/lps/lfc/events/LaszloEvents.lzs

Changeset: <http://svn.openlaszlo.org/openlaszlo/patches/20071220-maxcarlson-X.tar>http://svn.openlaszlo.org/openlaszlo/patches/20071220-maxcarlson-X.tar

Reply via email to