Looks great. Approved! On 2009-12-07, at 21:47, Henry Minsky wrote:
> Change 20091207-hqm-y by [email protected] on 2009-12-07 21:42:10 EST > in /Users/hqm/openlaszlo/trunk6 > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: fix for instance-mixin fails for non-top-level instance > > New Features: > > Bugs Fixed: LPP-8654 instance-mixin fails for non-top-level instance > > Technical Reviewer: ptw > QA Reviewer: max > Doc Reviewer: (pending) > > Documentation: > > Release Notes: > > Overview: > > > Details: > > + ViewCompiler: updateSchema recursively map over child elements > > + ClassModel: remove code in ClassModel that inserted a classdef into the DOM. > > + ScriptClass: added a "kind" field, for 'mixin', or 'class' > > + SWF9ParseTreePrinter: remove 'dynamic' modifier from mixin declaration, so > that > the interface it declares does not have 'dynamic' modifier on it. > > + updated lztest smoke check for mixins to have non-toplevel node with mixin > > Tests: > > test case from bug > test/mixins.lzx > smokecheck > test/smoke/mixin-simple.lzx > > Files: > M test/lztest/lztest-mixins.lzx > M test/smoke/mixin-simple.lzx > M WEB-INF/lps/server/src/org/openlaszlo/sc/SWF9ParseTreePrinter.java > M WEB-INF/lps/server/src/org/openlaszlo/sc/ScriptClass.java > M WEB-INF/lps/server/src/org/openlaszlo/compiler/ViewCompiler.java > M WEB-INF/lps/server/src/org/openlaszlo/compiler/ToplevelCompiler.java > M WEB-INF/lps/server/src/org/openlaszlo/compiler/ClassModel.java > > Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20091207-hqm-y.tar > > _______________________________________________ > Laszlo-reviews mailing list > [email protected] > http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews _______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
