[reSIProcate] RFC 3455 headers and parameters

[Dario Bozzali]

Hi Daniel,
I started looking at the code and, in my opinion, there is an asymmetry between ExtensionHeader and ExtensionParameter.
In ExtensionHeader constructor we use an assert if header name is not unknown, but the same check is not done in ExtensionParameter constructor even if we assume that parameter name is unknown when we use ExtensionParameter in ParserCategory.
I suggest to add a check in both constructors and throw an exception if header or parameter name is not unknown.
In ExtensionParameter we could add ExtensionParameter::Exception inner class (symmetric to ExtensionHeader::Exception).
I updated pull request at https://github.com/resiprocate/resiprocate/pull/6.
Regards,
Dario.

-----Original Message-----
From: Daniel Pocock [mailto:daniel at pocock.pro] 
Sent: venerdì 19 settembre 2014 10.20
To: Dario Bozzali
Cc: 'slgodin at gmail.com'; resiprocate-devel at resiprocate.org
Subject: Re: [reSIProcate] RFC 3455 headers and parameters

On 19/09/14 09:46, Dario Bozzali wrote:
> Hi Daniel,
> I created the separate pull request for project file.
> It can be consulted at the following link:
> https://github.com/resiprocate/resiprocate/pull/7

Great, I will let Scott look at that as he is using Windows heavily, my focus is mainly Linux

Dario, did you start making any change to the ExtensionHeader constructor or should I go and do that fix before pull request 6 is accepted?