https://bugzilla.novell.com/show_bug.cgi?id=402703

User [EMAIL PROTECTED] added comment
https://bugzilla.novell.com/show_bug.cgi?id=402703#c1





--- Comment #1 from Andy Hume <[EMAIL PROTECTED]>  2008-06-23 04:44:53 MDT ---
Created an attachment (id=223691)
 --> (https://bugzilla.novell.com/attachment.cgi?id=223691)
Patch

Patch attached.
1) Removes (the broken) lazy-initialization of the internal list.   Variously
broken: for instance not checked in the interface methods, and e.g.!
    if (this.panels == null)
      this.panels = new ArrayList (panels.Length);

2) Makes the IList interface members access the internal list only via the
equivalent public member and not directly -- where possible.  Thus now do the
correct StatusBarPanel initialization etc.

3) Fixed missing SetParent in set_Item[int].

4) Fixed Insert(int,StatusBarPanel), was 'replace' rather than 'insert'.

Hope it matches the coding standards...


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to