Mir

Code review comment for lp:~alan-griffiths/mir/mir-server-with-add_configuration_option-chaining

Revision history for this message
Andreas Pokorny (andreas-pokorny) wrote :

> Your argument that any clients that wants it could do it themselves is wrong:
> If there is a need to do it we should do it once and every one of these client
> will benefit.

agreed, lame argument. The thing that I left out was, that given your example the need or benefit is minimal. I believe most people will, unless we hilight its usage in big letters not even mind looking at how it has to be used.

I think you can invent or suspect a need for pretty much every imaginable feature.
So from that perspective, as the code exists and is mergeable we could simply merge it.

« Back to merge proposal