Re: [PATCH 4/4] strbuf.h: reorganize api function grouping headers

2014-12-12 Thread Jonathan Nieder
Jeff King wrote: > If you look at the original header file, these groupings actually did > exist (though we did not remotely follow them as functions were added), > and looked like: > > /* content related */ > > I'd be happy with something like that, too, but went with the style that >

[PATCH 4/4] strbuf.h: reorganize api function grouping headers

2014-12-12 Thread Jeff King
The original API doc had something like: Functions - * Life cycle ... some life-cycle functions ... * Related to the contents of the buffer ... functions related to contents etc This grouping can be hard to read in the comment sources, given the "