-----Original Message-----
From: Laszlo Teglas [mailto:[EMAIL PROTECTED]]
Sent: November 14, 2001 10:03 AM
To: Richard Bennett
Subject: RE: [Dynapi-Dev] dynimage patch
I'm not sure what the patch does, but to answer your second question:
DynImage.loadimages[i].img.width=(DynImage.loadimages[i].img.height=null);
is almost the same as:
DynImage.loadimages[i].img.width=null;
DynImage.loadimages[i].img.height=null;
If you want to be technical, it is actually:
DynImage.loadimages[i].img.height=null;
DynImage.loadimages[i].img.width=null;
Brackets and right side get evaluated first, but in this case order does not
really matter.
I personally prefer the following:
DynImage.loadimages[i].img.width=DynImage.loadimages[i].img.height=null;
(without the brackets)
Only for consistency, we seem to be using this style throughout the API.
The brackets are unnecessary and at first glance may confuse someone as a
function call.
NanoFace =:^)
-----Original Message-----
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED]]On Behalf Of Richard
Bennett
Sent: November 14, 2001 8:59 AM
To: [EMAIL PROTECTED]
Subject: [Dynapi-Dev] dynimage patch
Does anyone know what this patch is supposed to do, and if it should be
included?
https://sourceforge.net/tracker/?func=detail&aid=413030&group_id=5757&atid=3
05757
//next 3 lines are hook for IE, which stores pictures and picture sizes
//in different places and sets incorrect picture size if picture somewhy
//change it.
if (is.ie) {
DynImage.loadimages[i].img.width=(DynImage.loadimages[i].img.height=null);
DynImage.loadimages[i].complete = false;
DynImage.loadimages[i].img.src=DynImage.loadimages[i].origsrc=src;
}
and isn't this:
DynImage.loadimages[i].img.width=(DynImage.loadimages[i].img.height=null);
the same as this: ?
DynImage.loadimages[i].img.width=null;
DynImage.loadimages[i].img.height=null;
Richard.
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://www.mail-archive.com/[email protected]/
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://www.mail-archive.com/[email protected]/
