Re: Request for new round of SBuf review

Howdy,

As most of you aren't aware, Kinkie, alex and I had a bit of a
discussion about this on IRC rather than on the mailing list, so
there's probably some other stuff which should be posted here.

Kinkie, are you able to post some updated code + docs after our discussion?

My main suggestion to Kinkie was to take his code and see how well it
worked with some test use cases - the easiest and most relevant one
being parsing HTTP requests and building HTTP replies. I think that a
few test case implementations outside of the Squid codebase will be
helpful in both understanding the issues which this sort of class is
trying to solve.

I would really be against integrating it into Squid mainline until
we've all had a chance to play with it without being burdened by the
rest of Squid. :)

Adrian

2008/12/4 Kinkie <gkinkie_at_gmail.com>:
> Hi all,
> I feel that SBuf may just be complete enough to be considered a
> viable replacement for SquidString, as a first step towards
> integration.
> I'd appreciate anyone's help in giving it a check to gather feedback
> and suggestions.
>
> Doxygen documentation for the relevant classes is available at
> http://eu.squid-cache.org/~kinkie/sbuf-docs/ , the code is at
> lp:~kinkie/squid/stringng
> (https://code.launchpad.net/~kinkie/squid/stringng).
>
> Thanks!
>
> --
> /kinkie
>
>
Received on Sun Dec 07 2008 - 04:33:23 MST