Re: [Nmh-workers] about mhparam

[Valdis . Kletnieks]

On Tue, 14 Dec 2010 16:31:09 +0100, markus schnalke said:

> I improved the man page in some ways; see attached patch.
> 
> 
> I also spotted code that appears to be badly formated:
> --- a/uip/mhparam.c
> +++ b/uip/mhparam.c

Something that has proven to be very useful over in the Linux kernel world is
the concept of "One thing, one patch".  So this should be 2 separate patches,
one to fix the man page, and one to fix the code. The biggest reason for this
is that if it becomes necessary to revert the commit to the git tree, you
aren't reverting something unrelated along with it.

(Of course, a change to the code and matching man page updates should probably
be done as one commit).

pgpyWwTlKew2z.pgp
Description: PGP signature