Re: [dispatch] Two week review: Progressing draft-mohali-dispatch-originating-cdiv-parameter as AD sponsored

[[e-mail] (Dale R. Worley)]

Comments regarding the much-improved
draft-mohali-dispatch-originating-cdiv-parameter-03:

The ABNF in section 4.2 could be clarified and simplified, I think.
There are various nits I could mention, but those matters will be better
handled by the RFC Editor.

4.2.  ABNF

 sessioncase-param        = 1("sescase" EQUAL 1("orig" / "term")/ orig-cdiv)
 registration-state-param = "regstate" EQUAL 1("unreg" / "reg")
 orig-cdiv                = "orig-cdiv"

It seems like you could simplify the BNF by using "orig-cdiv" as a
constant rather than having a nonterminal for it.

The "1" multipliers aren't needed, since that is the default effect of
"(...)".  If the multiplers are removed, the outer parentheses for
"sessioncase-param" aren't needed.  Adding parentheses around the
first alternative of "sessioncase-param" makes the production clearer.
(per RFC 5234 section 3.5, "It is strongly advised that grouping
notation be used, rather than relying on the proper reading of "bare"
alternations, when alternatives consist of multiple rule names or
literals.")

The result of these changes would be

 sessioncase-param        = ("sescase" EQUAL ("orig" / "term")) / "orig-cdiv"
 registration-state-param = "regstate" EQUAL ("unreg" / "reg")

Dale