Re: [sipcore] Adam Roach's Discuss on draft-ietf-sipcore-reason-q850-loc-06: (with DISCUSS and COMMENT)

[Adam Roach <[e-mail]>]

The double-quotes are fine. Your RFC editor note addresses my issue 
completely. Clearing my discuss now.

/a

On 3/19/19 9:26 PM, Ben Campbell wrote:
>
>
>> On Mar 19, 2019, at 9:20 PM, Ben Campbell <[e-mail] 
>> <mailto:[e-mail]>> wrote:
>>
>>
>>
>>> On Mar 19, 2019, at 1:43 PM, Adam Roach <[e-mail] 
>>> <mailto:[e-mail]>> wrote:
>>>
>>> On 3/15/19 1:28 PM, Adam Roach wrote:
>>>> On 3/15/19 1:23 PM, Adam Roach wrote:
>>>>> On 3/11/19 3:23 AM,[e-mail] 
>>>>> <mailto:[e-mail]>wrote:
>>>>>> Hi,
>>>>>> Thank you for your comments. I went through the comments and here 
>>>>>> are my answers and proposals to the comments made.
>>>>>> Sorry If some of you get the mail twice, since I answered in 
>>>>>> another way already.
>>>>>>
>>>>>> 1.  Discuss:
>>>>>> §4 ABNF
>>>>>> Have incorporated the proposal.
>>>>>
>>>>>
>>>>> I'm going to clear my discuss
>>>>>
>>>>
>>>> Based on Paul Kyzivat's comment on the SIPCORE mailing list, I'm 
>>>> going to hold off and talk to Ben before I clear. Sorry for the 
>>>> confusion.
>>>>
>>>
>>> Based on Paul's comment and on Ben's reminder that SIP fields are 
>>> generally case-insensitive (both on the SIPCORE mailing list), I 
>>> think the solution here is to simply remove all of the instances of 
>>> "%s" in the BNF I sent. Sorry for the extra round-trip here.
>>>
>>
>> Like the following? If so, I can drop this into an RFC Editor note:
>
> Or do we also need to remove the double-quotes?
>
>>
>> OLD:
>> reason-extension =/ isup-cause-location
>> isup-cause-location =  "location" EQUAL isup-location-value
>>
>>     isup-location-value =
>>        %s"U" /      ; for 0 0 0 0 user
>>        %s"LPN" /    ; for 0 0 0 1 private network serving the local user
>>        %s"LN" /     ; for 0 0 1 0 public network serving the local user
>>        %s"TN" /     ; for 0 0 1 1 transit network
>>        %s"RLN" /    ; for 0 1 0 0 public network serving the remote user
>>        %s"RPN" /    ; for 0 1 0 1 private network serving the remote user
>>        %s"LOC-6" /  ; for 0 1 1 0 spare
>>        %s"INTL" /   ; for 0 1 1 1 international network
>>        %s"LOC-8" /  ; for 1 0 0 0 spare
>>        %s"LOC-9" /  ; for 1 0 0 1 spare
>>        %s"BI" /     ; for 1 0 1 0 network beyond interworking point
>>        %s"LOC-11" / ; for 1 0 1 1 spare
>>        %s"LOC-12" / ; for 1 1 0 0 reserved for national use
>>        %s"LOC-13" / ; for 1 1 0 1 reserved for national use
>>        %s"LOC-14" / ; for 1 1 1 0 reserved for national use
>>        %s"LOC-15"   ; for 1 1 1 1 reserved for national use
>>
>>                         Figure 1: isup-cause-location
>> NEW:
>> reason-extension =/ isup-cause-location
>> isup-cause-location =  "location" EQUAL isup-location-value
>>
>>     isup-location-value =
>>        "U" /      ; for 0 0 0 0 user
>>        "LPN" /    ; for 0 0 0 1 private network serving the local user
>>        "LN" /     ; for 0 0 1 0 public network serving the local user
>>        "TN" /     ; for 0 0 1 1 transit network
>>        "RLN" /    ; for 0 1 0 0 public network serving the remote user
>>        "RPN" /    ; for 0 1 0 1 private network serving the remote user
>>        "LOC-6" /  ; for 0 1 1 0 spare
>>        "INTL" /   ; for 0 1 1 1 international network
>>        "LOC-8" /  ; for 1 0 0 0 spare
>>        "LOC-9" /  ; for 1 0 0 1 spare
>>        "BI" /     ; for 1 0 1 0 network beyond interworking point
>>        "LOC-11" / ; for 1 0 1 1 spare
>>        "LOC-12" / ; for 1 1 0 0 reserved for national use
>>        "LOC-13" / ; for 1 1 0 1 reserved for national use
>>        "LOC-14" / ; for 1 1 1 0 reserved for national use
>>        "LOC-15"   ; for 1 1 1 1 reserved for national use
>>
>>                         Figure 1: isup-cause-location
>