Dear Michael,
Thank you for the encouragement and the tips.
You said:
> Rule #4: Pay D. Michael McIntyre $50,000
>
The check is in the mail. Keep a look out for it.
You said:
> Or just do the test in the calling code, like we talked
> about. That would be
> perfectly fine. We care about results first, and elegance
> isn't mandatory;
> though you should by no means feel dissuaded from pursuing
> elegance if you
> wish to do so.
Well, the error, in one fashion or another (for duplicate labels), already
exists in 10 files--taking a quick count. So, taking your advice and just
putting the status checking in would not change the current level of
readability, since I would just be swapping out one status check for another in
each file.
You also mention that this feature will not change much--that is true, too.
But if anyone else adds a feature that "requires" label suffixes, they will add
to the number of times this coding appears.
If, we are not in a hurry, I'd like to continue to explore "the elegant
approach." Chris has given some good advice, and I'm willing to continue at it
and learn along the way.
My current thought is to treat the use suffix labels similar to the
PropertyNames TransportMode and PropertyNames ZoomLevel found in
base/configuration.h & .cpp. They are accessible in base and stored in
KConfig. So the GUI can access them and so can base.
My STL is super weak, though. Give me some time.
Also, I'm signing off for the weekend. Got lots of family and friend time to
put in. I'll be back at it Monday.
Sincerely,
Julie S.
-------------------------------------------------------------------------
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
