Re: Request for new round of SBuf review

From: Kinkie <>
Date: Sun, 7 Dec 2008 21:26:38 +0100

Hi all,
 Not much new code yet, but as a result of our discussion there's a
small adjustmdent in strategy, which in the short term has two parts:
the code I posted is valid and I still wish it to be reviewed. SBuf 's
purpose changes as it'll aim at catering the need to handle blobs
I've set off writing StringNg which uses a Sbuf as storage and
reexports most of its interface, trying to keep track of encodings
In practical terms , Sbuf:.cmpNoCase moves to String. As soon as it's
ready and while SBuf's review is pending, I'll follow Adrian's suggestion and
try to prototype a parser using the unit tests infrastructure.

Written while walking in the cold @6am , please excuse monstrousities
(including not addressing to the dev ml, reposting the reply with a
few spelling adjustments)


On Sun, Dec 7, 2008 at 5:33 AM, Adrian Chadd <> wrote:
> 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 <>:
>> 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
>> , the code is at
>> lp:~kinkie/squid/stringng
>> (
>> Thanks!
>> --
>> /kinkie

Received on Sun Dec 07 2008 - 20:26:43 MST

This archive was generated by hypermail 2.2.0 : Thu Dec 11 2008 - 12:00:03 MST