klistviewsearchline

Daniel Teske teske at squorn.de
Sun Jul 3 17:52:57 BST 2005


> > I think it's correct.
> I think so too. 
Good

> Comments on the patch follow.
Thanks.

> Don't check this in.
Right

> Or this.
Errf, I thought, I had cleaned up.

> This line is redundant, since findFirstHiddenToVisible does this
> anyway. It's confusing to have this here, since it looks like it
> makes all the children of item visible (whereas it actually does
> nothing).
Right.

> Is this supposed to be part of the same patch? If so, can you explain
> it?
That's a fix for a different bug. Sorry for the confusion.

daniel




More information about the kde-core-devel mailing list