Come to think of it, the line is not correct anyway.
If the item is selected then there is no reason to set it.
if (this.selected==b) return;
--
// Robert Rainwater
On 1/30/2001, 1:28:49 PM EST, Doug wrote about "[Dynapi-Dev] [Bug #130357] small typo
in list.js":
> That just plain silly.
> It's perfectly valid coding technique.
> perhaps it would be safer to assign the value first and then test the condition...
> -----Original Message-----
> From: [EMAIL PROTECTED]
>[mailto:[EMAIL PROTECTED]]On Behalf Of Michael Pemberton
> Sent: 30 January 2001 13:28
> To: [EMAIL PROTECTED]
> Subject: Re: [Dynapi-Dev] [Bug #130357] small typo in list.js
> please put the "typo" back. What is does is set the value of this.selected to b
>and then evaluates the returned value (the new value of this.selected). It is needed
>to change the selected
> state of the item.
> [EMAIL PROTECTED] wrote:
> Bug #130357, was updated on 2001-Jan-28 14:27
> Here is a current snapshot of the bug.
> Project: DynAPI 2
> Category: Core - Widgets
> Status: Closed
> Resolution: Fixed
> Bug Group: None
> Priority: 5
> Submitted by: marstr
> Assigned to : rainwater
> Summary: small typo in list.js
> Details: there's a small typo (i think) in list.js line 76
> if (this.selected=b) {
> should be
> if (this.selected==b) {
> right?
> Follow-Ups:
> Date: 2001-Jan-28 17:50
> By: rainwater
> Comment:
> Doh! Its been updated in CVS. Closing this bug.
> -------------------------------------------------------
> For detailed info, follow this link:
> http://sourceforge.net/bugs/?func=detailbug&bug_id=130357&group_id=5757
> _______________________________________________
> Dynapi-Dev mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/dynapi-dev
> --
> Michael Pemberton
> [EMAIL PROTECTED]
> ICQ: 12107010
----------------------
DynAPI Snapshots: http://dynapi.sourceforge.net/snapshot/
DynAPI Homepage: http://dynapi.sourceforge.net/
_______________________________________________
Dynapi-Dev mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/dynapi-dev
