On Fri, 07 Nov 2003 02:17:02 -0600
Doug Quale <[EMAIL PROTECTED]> wrote:
[...]
many thanks for your really professional revision. I learned a lot while studying it.
Your addition of the collapse/expand feature is splendid. only I must admit I do not
yet understand thoroughly whats going on there.
[...]
> Pygtk already includes the method TreeModel.is_ancestor() that does
> the same thing as checkSanity().
now, thats the only point at which I'm dissenting with you. "sanity" is in that case
not alone a question of ancestry, but of identity too. (try dropping an iter onto
itself with your version). One could think that replacing the line:
if not model.is_ancestor(source, target):
with:
if not model.is_ancestor(source, target) and not (source == target):
should do the job. but no, it doesn't work. for some reasons, buried deep in
gtk/pygtk, the two objects are never identic, even if they are "the same". therefore I
still see the best solution in comparing the paths.
[...]
have a nice weekend.
walter
_______________________________________________
pygtk mailing list [EMAIL PROTECTED]
http://www.daa.com.au/mailman/listinfo/pygtk
Read the PyGTK FAQ: http://www.async.com.br/faq/pygtk/
