Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-20 Thread Alex Rousskov
On 08/20/2015 07:13 AM, Amos Jeffries wrote: This part of the email thread was discussing whether the existence of admin scripts (rather than various imprecise syntax rules or cachemgr.cgi code) should be the primary factor in our decision making. How is the above information relevant to that

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-20 Thread Amos Jeffries
On 20/08/2015 9:18 a.m., Alex Rousskov wrote: On 08/19/2015 09:47 AM, Amos Jeffries wrote: On 19/08/2015 4:50 p.m., Alex Rousskov wrote: On 08/15/2015 12:20 AM, Amos Jeffries wrote: For now this class is specifically and intentionally dumping out the (old) format for cachemgr.cgi. Other

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-19 Thread Alex Rousskov
On 08/19/2015 09:47 AM, Amos Jeffries wrote: On 19/08/2015 4:50 p.m., Alex Rousskov wrote: On 08/15/2015 12:20 AM, Amos Jeffries wrote: For now this class is specifically and intentionally dumping out the (old) format for cachemgr.cgi. Other third-party tools are considered only so far as the

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-18 Thread Alex Rousskov
On 08/15/2015 12:20 AM, Amos Jeffries wrote: I dont like payload particularly either in this case. But page is wrong. Page is what the remote end display tool will be generating. *IF* a page exists at all. There are many kinds of payloads. There are many kinds of pages. There are many kinds

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-15 Thread Amos Jeffries
New patch attached. But only some of the details are changed. We have some pretty significant items not yet agreed on. On 13/08/2015 10:19 a.m., Alex Rousskov wrote: On 08/12/2015 06:28 AM, Amos Jeffries wrote: On 12/08/2015 5:07 a.m., Alex Rousskov wrote: On 08/10/2015 08:20 AM, Amos

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-12 Thread Alex Rousskov
On 08/12/2015 06:28 AM, Amos Jeffries wrote: On 12/08/2015 5:07 a.m., Alex Rousskov wrote: On 08/10/2015 08:20 AM, Amos Jeffries wrote: Here is mk2 of the Formatter class for doing display things to CacheMgr report payloads. Please post the patch (your post had no attachments), preferably

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-12 Thread Amos Jeffries
On 12/08/2015 5:07 a.m., Alex Rousskov wrote: On 08/10/2015 08:20 AM, Amos Jeffries wrote: Here is mk2 of the Formatter class for doing display things to CacheMgr report payloads. Please post the patch (your post had no attachments), preferably reflecting the discussion that happened since

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-11 Thread Alex Rousskov
On 08/10/2015 08:20 AM, Amos Jeffries wrote: Here is mk2 of the Formatter class for doing display things to CacheMgr report payloads. Please post the patch (your post had no attachments), preferably reflecting the discussion that happened since then. Thank you, Alex.

Re: [squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-10 Thread Amos Jeffries
On 11/08/2015 2:20 a.m., Amos Jeffries wrote: Here is mk2 of the Formatter class for doing display things to CacheMgr report payloads. I have removed the ostream parts of Alex proposal. Since these formatter could create StoreEntryStream at need if they really had to and the formatting API

[squid-dev] [PATCH] PayloadFormatter (was PackableStream)

2015-08-10 Thread Amos Jeffries
Here is mk2 of the Formatter class for doing display things to CacheMgr report payloads. I have removed the ostream parts of Alex proposal. Since these formatter could create StoreEntryStream at need if they really had to and the formatting API does not require that functionality. I've labelled