On 23/07/06, Chris Cannam <[EMAIL PROTECTED]> wrote:
> Stephen Torri:
> > Legal as in C++ compilers allow. Illegal if you want code that is
> > maintainable others.
>
> Ooh, you've got me on the ropes!
>
> Did either of you actually have to spend a significant
> amount of time to work out what that syntax did, or
> is the problem just that it looks weird?
>
> Maybe I am coming from a position of over familiarity
> with it.

It made perfect sense to me, but that's probably not a mark of
respectability. :)

I would be tempted to replace the nested expression with an inline
member function
getThingummyColour() of the params struct, so it ends up looking like this:

NoteCharacter body = getCharacter
       (charName, params.getThingummyColour(), inverted);

..where "thingummy" is replaced by whatever makes sense. If you define
the inline function right there'll be no performance hit either.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Rosegarden-devel mailing list
[email protected] - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel

Reply via email to