On Tuesday 22 July 2008, Julie S wrote:

> At this point, I'm pretty confident that the code for this patch must not
> reside in any base code at all.

Seems like a sensible conclusion.

> My though is to treat this like misc/Strings.  I'd like to move the code to
> misc/LabelSuffixes.  That way I can use KConfig and avoid and src/base
> changes.
>
> Any thought on this would be nice.

I can't think of any better place to put it off hand, and having taken a 
glance at some of the places where this would need to be used, encapsulating 
the whole bit into some tidy call to something somewhere would definitely be 
easier to read and more maintainable.

I think I might just throw this directly into misc/Strings.* myself, since 
this is going to be a small amount of code, and since every affected file 
should already include misc/strings.h anyway.  That way you don't have to add 
new files to the build system, and add new includes everywhere just for this 
one small bit.
-- 
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