On Saturday 12 Feb 2005 20:08, Pedro Lopez-Cabanillas wrote:
> I'm not going to commit the fix without your approval, but this
> QUERY_PLUGINS_FROM_GUI thing seems a feature to me, not a bug fix.

It's an optimisation, not a feature.  The code that's run in the GUI if 
QUERY_PLUGINS_FROM_GUI is set was run in the sequencer before, but this 
way we eliminate a big load of comms on startup.

If something called from the QUERY_PLUGINS_FROM_GUI code causes 
subsequent locale dependency problems, then without 
QUERY_PLUGINS_FROM_GUI those problems will simply occur at the 
sequencer instead of the GUI  (the sequencer also does potentially 
locale-dependent number/string conversions), so backing out 
QUERY_PLUGINS_FROM_GUI will not be a safe general fix, just another 
workaround for special cases in the GUI.  (Try printing out the decimal 
separator value from rosegardensequencer.cpp:1415, for example.)

I think the only option really is a proper fix -- find out what's 
causing the problem and stop it doing it.


Chris


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Rosegarden-devel mailing list
[email protected] - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel

Reply via email to