[Laszlo-dev] <handler> implementation for review

[Sarah Allen]

Sorry I didn't read this proposal earlier.  Overall, it seems like an 
improvement.  I don't think this affects your changset, but it is 
feedback to another part of the proposal.

I was thinking that this idiom:

 <event name="eventName"/>
 <handler name="eventName" [reference="..."] method="methodName" />

 <method name="methodName">
   ...
 </method>

would be clearer as:

 <event name="eventName" method="methodName"/>

 <method name="methodName">
   ...
 </method>

or am I missing something?

Sarah

Henry Minsky wrote:

> Summary: Add <handler> and <event> support to nodes and classes
>
> New Features: <handler> and <event> tags
>
> Bugs Fixed:
>
> Technical Reviewer: ptw
>
> QA Reviewer: mdavis
>
> Doc Reviewer:
>
> Documentation:
>
> Release Notes:
>
> Details:
>
> Implements subset of proposal at
> http://wiki.openlaszlo.org/Event_and_handler_tags
>
> There are currently no compiler warnings for the case where a handler
> is declared for an undeclared event (in the current class or
> superclass).
>
> This implementation adds <handler> and <event> to the schema, and
> basically forks a copy of the existing compiler support for <method>
> to implement <handler>.
>
> The <event> tag is treated identically to an <attribute>
> declaration. For <event> tags in <class> declarations, eventually the
> compiler should make a list of events declared on a class, so that
> when a <handler> is encountered, the compiler can check the superclass
> chain to make sure that the specified event is declared on the class.
>
> [It will be harder to try to check this at compile time if the
> <handler> has a 'reference' attribute]
>
> Unit test in test/events/handler.lzx, incomplete test in warnings.lzx
>
>
> -- 
> Henry Minsky
> Software Architect
> hminsky at laszlosystems.com <mailto:hminsky at laszlosystems.com>
>
>------------------------------------------------------------------------
>
>_______________________________________________
>Laszlo-dev mailing list
>Laszlo-dev at openlaszlo.org
>http://www.openlaszlo.org/mailman/listinfo/laszlo-dev
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://openlaszlo.org/pipermail/laszlo-dev/attachments/20060110/97729745/attachment.html