Messages in this thread | | | From | "Luck, Tony" <> | Subject | RE: [PATCH v9 3/3] aerdrv: Cleanup log output for AER | Date | Fri, 4 Jan 2013 22:22:09 +0000 |
| |
I sent a "please pull" to Ingo/Peter/Thomas about an hour ago ... if they push back (or ignore) we can fold your ack and nit-picks into another version.
> s/elimiating/eliminating/ above. Ugh ... nobody spotted this one ("many eyes" really does work!)
> I remove the "v1-v2" notes when I merge patches because I don't think > they're useful any more. But if Tony applies these, he can use his > judgment.
Yes - I do too. In fact it is easier on the maintainer if this sort of meta-commentary goes *after* the "---" in the patch. Then it is available for review (where it is most helpful), but tools will automatically drop it when applying the patch.
-Tony
| |