On Tuesday 22 July 2008, Chris Cannam wrote:
> No, that part at least is simple (thanks to Pedro's fine work).  Just
> add it to the relevant src/*FileList.txt (in this case
> MiscFileList.txt) and re-run cmake.

True.  Adding files is no big deal.  I'm just thinking it would be one less 
complexity to deal with when we merge all of this into the port, and the 
amount of code "polluting" Strings.* would be small, and its status as 
pollution somewhat debatable, since this new code logically stems from the 
whole annoying avoiding-QString-in-base problem in the first place.

> Question: Does this all leave you thinking "This would all be so
> simple if only we were using the Qt/KDE classes in base?" or just
> thinking "This is evidently a weird corner case, and it's frustrating
> that it doesn't seem easy to work around"?

For my part, I do think her original idea was cleaner and easier to follow.  
We've thrown a hurdle in the way for little good purpose overall, and the 
whole practice in question is arguably pointless.  However, removing the 
hurdle at this juncture would be even more trouble than merely working around 
it one more time.  It seems like she's about got it figured out, and that 
means we can avoid refactoring bloody everything to lift the ban on QT in 
base/, so that might be the best course to pursue, in light of all the other 
challenges we have to deal with just now.
-- 
D. Michael McIntyre 

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Rosegarden-devel mailing list
[email protected] - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel

Reply via email to