Mir

Code review comment for lp:~robert-ancell/mir/frontend-class-renaming

  1. frontend-class-renaming
  2. Merge into trunk
Revision history for this message
Robert Ancell (robert-ancell) wrote :

> We've a rule in the codebase that we don't have null dependencies (and use
> null objects instead). I don't think we should be making an exception here
> just to avoid having a trivial NullMessageProcessor class.

I wasn't aware that is the case, I'll work on fixing that. Can you please add this information to the style guide?

« Back to merge proposal