[Laszlo-dev] For Review: Change 20080502-ptw-H Summary: Make handler trampoline work in swf9

[Henry Minsky]

Aren't the override keywords supposed to be inserted automatically ?
Do we need to declare toString on something in the lfc.lzx schema file so
that the compiler
knows to insert an override?

On Fri, May 2, 2008 at 5:18 PM, P T Withington <ptw at pobox.com> wrote:

> Well toString has to be a public override for it to work, so, I think the
> example will have to be updated.
>
>
> On May 2, 2008, at 14:09, Max Carlson <max at openlaszlo.org> wrote:
>
>  Your example works, but
> > http://localhost:8080/trunk/examples/modeexample.lzx?lzr=swf9 now gives
> > a compiler error:
> > org.openlaszlo.sc.CompilerError: line 366: Error: Incompatible override,
> > in line: override function toString () {
> >
> > P T Withington wrote:
> >
> > > Change 20080502-ptw-H by ptw at dueling-banjos.home on 2008-05-02
> > > 06:48:32 EDT
> > >   in /Users/ptw/OpenLaszlo/ringding-2/WEB-INF/lps/lfc
> > >   for http://svn.openlaszlo.org/openlaszlo/trunk/WEB-INF/lps/lfc
> > > Summary: Make handler trampoline work in swf9
> > > Bugs Fixed:
> > > LPP-5818 'Warn when methods registered for events aren't expecting
> > > exactly one argument from sendEvent()'
> > > Technical Reviewer: hminsky (pending)
> > > QA Reviewer: max (pending)
> > > Details:
> > >   LzFormattter: Type and default args for swf9, work around switch
> > >   verifier error (must have break on final case).
> > >   *: toString must be public to have it seen by string coercion
> > > Tests:
> > >   This now runs in swf9, demonstrating that the wrapper to handle
> > >   the missing argument is being installed.
> > >   <canvas>
> > >     <simplelayout />
> > >     <text name="output" clickable="true" text="Click Me!"/>
> > >     <method name="handleMe">
> > >       output.format("Clicked!");
> > >     </method>
> > >     <handler name="onclick" reference="output" method="handleMe" />
> > >   </canvas>
> > > Files:
> > > M      kernel/swf/LzLibrary.lzs
> > > M      kernel/dhtml/LzLibrary.js
> > > M      services/LzURL.js
> > > M      services/LzIdle.lzs
> > > M      debugger/LzMemory.lzs
> > > M      core/LzMiniNode.lzs
> > > M      core/LzNode.lzs
> > > M      views/LzText.lzs
> > > M      views/LaszloCanvas.lzs
> > > M      helpers/LzFont.lzs
> > > M      helpers/LzSelectionManager.lzs
> > > M      events/LaszloEvents.lzs
> > > M      controllers/LzAnimatorGroup.lzs
> > > M      controllers/LaszloLayout.lzs
> > > M      controllers/LaszloAnimation.lzs
> > > M      data/LzLazyReplicationManager.lzs
> > > M      data/LzReplicationManager.lzs
> > > M      data/LzDatapointer.lzs
> > > M      data/LzDataText.lzs
> > > M      data/LzDataElement.lzs
> > > M      data/LzDataset.lzs
> > > M      data/LzDatapath.lzs
> > > M      data/LzResizeReplicationManager.lzs
> > > M      data/LzParsedPath.lzs
> > > M      data/LzParam.lzs
> > > M      data/LzDataAttrBind.lzs
> > > M      compiler/LzFormatter.lzs
> > > Changeset:
> > > http://svn.openlaszlo.org/openlaszlo/patches/20080502-ptw-H.tar
> > >
> >
> > --
> > Regards,
> > Max Carlson
> > OpenLaszlo.org
> >
>


-- 
Henry Minsky
Software Architect
hminsky at laszlosystems.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.openlaszlo.org/pipermail/laszlo-dev/attachments/20080502/1ea2c887/attachment.html