Re: [PATCH] protocol_t upgrade

 On Mon, 28 Feb 2011 18:23:51 -0700, Alex Rousskov wrote:
> On 02/28/2011 04:44 PM, Amos Jeffries wrote:
>> In the final URL and protocol design we have AnyP::ProtocolType
>> which
>> enumerates the registered protocols accepted in URLs (lower case)
>> and
>> the request protocol field (upper case). Then URLScheme which
>> expands
>> that to allow other protocol names accepted in URLs.
>
> Perhaps this is too abstract to allow a meaningful discussion right
> now,
> but why not simplify and just have one AnyP::Protocol class that
> stores
> the ID and the image of any protocol? Can that cover both URL scheme
> (lowercase) and MIME message first-line (upper case) uses, among
> other
> things?

 I'm build-testing a patch which includes the missing changes to
 URLScheme which does the split nicely without much alteration to
 existing code.
 Will have that for you in a few hours.

 For now there are places using local representations in strings etc of
 the protocol, or using protocol_t where it should be URLScheme. Future
 polish will clear those up correctly once this patch straightens out the
 two. Then we can move on to supporting unknown URL protocols with
 pass-thru.

 Amos
Received on Tue Mar 01 2011 - 01:50:39 MST