Re[2]: [Dynapi-Dev] [Bug #130357] small typo in list.js

[Robert R. <rra...@ya...>]

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: dyn...@li... [mailto:dyn...@li...]On Behalf Of Michael Pemberton
>     Sent: 30 January 2001 13:28
>     To: dyn...@li...
>     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. 
>     no...@so... 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 
>       Dyn...@li... 
>       http://lists.sourceforge.net/lists/listinfo/dynapi-dev

>     -- 
>     Michael Pemberton 
>     mp...@ph... 
>     ICQ: 12107010 
      



----------------------
DynAPI Snapshots: http://dynapi.sourceforge.net/snapshot/
DynAPI Homepage: http://dynapi.sourceforge.net/