Hi Fred,

On 2004-02-14 14:11:51 +0100 Fred Kiefer <[EMAIL PROTECTED]> wrote:

I did (hopefully) resolve this problem in a different way, by removing the helper method and inlining the code into reloadItem:reloadChildren:. This allowed for a few more corrections/simplifications (and would allow for more, if I only would understand better, what this method should do). I must admit, that NSOutlineView is a part of GUI, where I don't understand most of the code. No idea, for example, why it replaces the item with NULL, if it is nil.

I just had a look at what you checked in two days ago. Seems ok to me, but a part of my patch is missing, and I wonder why.
At the end of the reloadItem... method I updated _items and _levelOfItems to ensure that those containers reflect the possible change of the item object. Did you just forget this or was something wrong with that part of the patch?
Otherwise, thanks a lot for checking this in.


Andreas

--
Andreas Heppel

Mail: aheppel at web dot de
Home: http://www.andreasheppel.de

Check out Burn.app - the CD burning frontend for GNUstep
http://gsburn.sourceforge.net



_______________________________________________
Bug-gnustep mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/bug-gnustep

Reply via email to