On May 16, 2006, at 8:02 AM, John Sundman wrote: > On May 16, 2006, at 10:42 AM, Amy Muntz wrote: > >> Task 2 >> Then, please use the DeclareEvent pattern, and remove the if's as >> you no longer have to check for null. Please do this in the same >> set of files you are currently working with, and then send that >> out for review (Henry has volunteered to review this next >> changeset - thanks, Henry!) > > 1) Will we be scrubbing the components as well? (Using the > declarative <event> tag instead of DeclareEvent, presumably?) I > believe that this has been done for some components, but not all > components. > All of the components have been converted to use <event>. This was done for 3.2.0. However, there may still be events that are used but not declared.
> 2) Presumably I should at some point go over all example code in > the docs to make sure to remove all checks for null events. I > have just converted all event handlers to the new syntax, but I > didn't remove the null checks. I don't suppose that this is a > very high priority, but I do think I should proably do this before > legals ships, so that people don't develop any bad habits. > See my reply to the other thread. We can't just remove the null checks -- we need to substitute a SendEvent(evt, args) function for the "if (evt != null) evt.sendEvent(arg)" idiom. > Comments? > > jrs > _______________________________________________ Laszlo-dev mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-dev
