On Tue, Sep 04, 2007 at 11:46:59PM +0000, Ren=C3=A9 Scharfe wrote:
quoted text
> Pierre Habouzit schrieb:
> >   This is just cleaner way to deal with strbufs, using its API rather t=
han
quoted text
> > reinventing it in the module (e.g. strbuf_append_string is just the pla=
in
quoted text
> > strbuf_addstr function, and it was used to perform what strbuf_addch do=
es
quoted text
> > anyways).
> > ---
> >  archive-tar.c |   65 ++++++++++++++-----------------------------------=
--------
quoted text
> >  1 files changed, 16 insertions(+), 49 deletions(-)
>=20
> 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.

--=20
=C2=B7O=C2=B7  Pierre Habouzit
=C2=B7=C2=B7O                                                madcoder@debia=
n.org
OOO                                                http://www.madism.org