[Laszlo-dev] DeclareEvent(s) going away
Henry Minsky
henry.minsky at gmail.com
Mon Nov 5 12:04:38 PST 2007
build-tools/runlzunit.sh tries to run the browser like this, it
appears to be trying to get firefox launched , if your name is Darwin?
case "$hosttype" in
Darwin)
build_os=osx
[ -n "${kill}" ] && killall firefox-bin
browser=firefox
function cygpath () { echo $1; }
;;
Linux)
build_os=unix
browser=firefox
function cygpath () { echo $1; }
;;
CYGWIN*)
build_os=windows
browser="/cygdrive/c/Program Files/Mozilla Firefox/firefox.exe"
[ -n "${kill}" ] && kill -9 `ps -Ws | grep -i firefox | awk '{print $1}'`
echo "browser = ${browser}"
;;
*)
exit 1
;;
esac
On 11/5/07, P T Withington <ptw at openlaszlo.org> wrote:
> Hey wow. I just tried that, and it launched parallels! Is this
> somehow trying to run ie6 on my mac or what?
>
> On 2007-11-05, at 13:55 EST, Benjamin Shine wrote:
>
> >
> > Wow. Massive.
> >
> > Phil, I see you're the QA reviewer. Can you please run "ant
> > runlzunit" before checking it in? (That does lztest plus runs a lot
> > of lzunit tests in the browser in both runtimes.)
> >
> > Once this is checked in, I think the interactive demos should be
> > given a once-over by QA; this change touches so much code that it is
> > more likely to contain problems, despite the impeccable quality of
> > Tucker's code.
> >
> > -ben
> >
> >
> >
> > On Nov 5, 2007, at 8:17 AM, P T Withington wrote:
> >
> >> I am about to check in a change that removes DeclareEvent and
> >> DeclareEvents. These were private functions, so no API change
> >> review is needed. I am removing these to support my fix for
> >> LPP-4997, but also because these functions were dynamically adding
> >> attributes to a class at run time, which impacts performance in
> >> JS2. If you are working in the LFC or writing a class in <script>
> >> you will no longer say:
> >>
> >> class ... {
> >> ...
> >> DeclareEvent(prototype, 'onSomething');
> >> ...
> >> }
> >>
> >> Instead you should simply declare the event property like any other
> >> instance variable, but give it a proper initial value:
> >>
> >> class ... {
> >> ...
> >> var onSomething = LzDeclaredEvent;
> >> ...
> >> }
> >>
> >> Eventually, we will want to give these proper type declarations (as
> >> soon as Don gives us the go-ahead). Events are a non-nullable
> >> type. Perhaps we should change the way you specify a null event
> >> though? Something like:
> >>
> >> class ... {
> >> ...
> >> var onSomething:LzEvent = LzEvent.nullEvent;
> >> ...
> >> }
> >>
> >> Thoughts?
> >>
> >> ---
> >>
> >> Note: There are no changes to the usage of LZX <event> tag. The
> >> compiler will automatically translate the <event> tag to the new JS
> >> format.
> >
>
>
--
Henry Minsky
Software Architect
hminsky at laszlosystems.com
More information about the Laszlo-dev
mailing list