* Sébastien MARQUE -- 2/20/2009 1:23 AM:
> This is not new, it was the case with the precedent gui dialog, and I 
> used a Nasal workaround but not really useful as it disallows to come 
> back to a previous state.

I've written a lot of dialogs, the most complicated ones among them,
and I don't remember that I ever had to work around this sort of issue.
That's one of the reasons why I hesitate when someone asks for a change
to make his first(?) dialog work.  :-P

But anyway. I withdraw my objection and don't mind if someone commits
this patch with one improvement: the node value shall not be read twice!
That's not so much because of the saved nanoseconds, but mostly because
of --trace-read (and style).

dialog-apply does then no longer mean copy widget data to the tree,
but sync tree with the internal widget state. (dialog-update does still
mean *copy*, though ... which is another reason why I'm not thrilled
by the change. One more inconsistency to remember ...)

But be warned: if we run into problems with this modification, I'll be
less hesitant before throwing it out again.  ;-)

m.

------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/flightgear-devel

Reply via email to