[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