Subject: [Ietf-calsify] Inconsistent BNF for "the following is optional and may occur more than once"

From: Nigel.Swinson at rockliffe.com (Nigel Swinson)
Date: Wed Dec 13 06:27:18 2006

Minor nits:

BNF for DTDue says:
http://tools.ietf.org/html/draft-ietf-calsify-rfc2445bis-03#section-3.8.2.3
        dueparam   = *(
+
                   ; the following are optional,
                   ; but MUST NOT occur more than once

                   (";" "VALUE" "=" ("DATE-TIME" / "DATE")) /
                   (";" tzidparam) /

                   ; the following is optional,
                   ; and MAY occur more than once
-                   *(";" xparam)
+                  (";" xparam)

                   )

BNF for DTStart says:
http://tools.ietf.org/html/draft-ietf-calsify-rfc2445bis-03#section-3.8.2.4
        dtstparam  = *(

                   ; the following are optional,
                   ; but MUST NOT occur more than once

                   (";" "VALUE" "=" ("DATE-TIME" / "DATE")) /
                   (";" tzidparam) /

                   ; the following is optional,
                   ; and MAY occur more than once

-                     *(";" xparam)
+                    (";" xparam)

                   )

The *(";" xparam) bit is I believe unintentionally inconsistent with the other values like DTEnd and I think the * should be dropped relying instead on the top level *() of the param token.  Also there's a missing newline in the dueparam making the formatting inconsistent with the others.

Cheers

Nigel