On Tue, Sep 04, 2007 at 11:46:59PM +0000, René Scharfe wrote: > Pierre Habouzit schrieb: > > This is just cleaner way to deal with strbufs, using its API rather than > > reinventing it in the module (e.g. strbuf_append_string is just the plain > > strbuf_addstr function, and it was used to perform what strbuf_addch does > > anyways). > > --- > > archive-tar.c | 65 ++++++++++++++------------------------------------------- > > 1 files changed, 16 insertions(+), 49 deletions(-) > > Apart from the wrong subject line I really like this patch. :-D oh boy, yes I fixed that in my local patch collection. I'm waiting for a few hours (days ?) to see if there will be some more comments, I've integrated every single one done here already (and some I had on IRC too), and I'll repost a new clean series that I intend to be a real proposal for inclusion. And yes, this patch is a perfect example of the gain we have to share a common buffer API. The code looks (at least to me) way nicer, and if you look in the details, we perform as many memory allocations, copies, and so on as in the previous version. -- ·O· Pierre Habouzit ··O madcoder@xxxxxxxxxx OOO http://www.madism.org
Attachment:
pgpB9KpC77ULM.pgp
Description: PGP signature
