[lvm-devel] LVM2 tools/lvmcmdline.c liblvm/lvm_base.c lib/ ...

Alasdair G Kergon agk at redhat.com
Sat May 7 18:24:13 UTC 2011


On Sat, May 07, 2011 at 01:50:19PM -0000, mornfall at sourceware.org wrote:
> +	/* Set in/out stream buffering before glibc */

Please add more explanatory comments inline.

I mentioned this on IRC but there's been a trend towards not giving
explanations inline which I'd like to see reversed.  Patch headers
are becoming too long, containing information about the new version
of the code which should instead have been placed inline.

Alasdair




More information about the lvm-devel mailing list