On Sun, Oct 01, 2017 at 04:45:45PM +0200, René Scharfe wrote: > strbuf_addf() can be used to add a specific number of space characters > by using the format "%*s" with an empty string and specifying the > desired width. Use strbuf_addchars() instead as it's shorter, makes the > intent clearer and is a bit more efficient. Nice. The result looks much clearer to me. (I won't respond individually to the coccinelle transform patches you posted earlier, but they also all look obviously correct to me). -Peff
