// COPY ///
By the way, if you want to reduce code and you code a "proper" way
(I'm sure that's what you do), you can go for hunting any line like
the one below (the last one you sent, for setWidth I guess),
> if (noevt!=false) this.invokeEvent('resize');
and replace with less elegant things like
if (!noevt) this.invokeEvent('resize');
"much" shorter.
/// END COPY ///
Unfortunately you have just changed the logic of that statement.
If someone passes false as a parameter, your code WILL invoke an
event call, but the above code WILL NOT invoke an event call.
On the other hand if nothing (or null) is passed, your code WILL
invoke an event where the above code WILL NOT invoke an event. (null
RESOLVES AS false when tested logically but IS NOT EQUAL to false
when tested against it.
To pound this home:
Passed (!noevt) (noevt != false) (passed)
true false true true
false true false false
null true true false
The code as written by the dynAPI people is equivalent to: unless the
user ACTIVELY cancels the event (by passing FALSE) it will occur.
(this variable is poorly named; "doEvent" would be a bette name for
the variable)
Your code reads: unless the user cancels the event (by passing TRUE)
the event will occur.
Expect annoying hard to track errors if you change the logic from the
default. ALSO THINK REALLY HARD before changing other people's code!
=====
=======================
'Providing year 2001 consultations at rates as low as $5,000 an hour.'
Dave Edelhart
Director of Operations, Manatee Bay Productions
www.manateebay.com
[EMAIL PROTECTED]
__________________________________________________
Do You Yahoo!?
Get personalized email addresses from Yahoo! Mail
http://personal.mail.yahoo.com/
_______________________________________________
Dynapi-Help mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dynapi-help
