Albrecht Schlosser wrote:
> On 11.07.2010, at 20:28, Greg Ercolano wrote:
>>      The tree.fl will need the final fixes, and at some point tree.cxx will
>>      need to be removed from svn and replaced with just tree.fl.
> 
> Greg, please be careful with tree.cxx. You're looking at a stale svn 
> copy. As I wrote in my other post already, Matt *changed* tree.fl, and
> I *removed* tree.cxx and tree.fl already !!!

        Ah good (I take you meant tree.h was removed, and not tree.fl)

        Just checked out a fresh FLTK and I see the new tree.fl, so I'll
        make a new patch against that just to see if all makes sense.

        New patch will have the remove(Fl_Widget*/&) override.

        Possibly other Fl_Group methods will need overriding too,
        would need to check more carefully another time, as I have
        to go back into work mode on Monday.

> one more reason to make
> Fl_Group::remove() virtual and then implement it in Fl_Tree to catch
> widget removals.

        I should at least be able to add remove() to Fl_Tree now,
        and it should operate OK using normal override rules.

        So later, should Fl_Group::remove() become virtual, that
        should be OK too.
_______________________________________________
fltk-dev mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk-dev

Reply via email to