Hi,

I like the idea of changing the production to:

comment = LPAREN *(ctext / quoted-pair / comment) SWS ')' ; case 1

however changing "LPAREN" and "RPAREN" to:

LPAREN = "(" SWS ; case 2
RPAREN = SWS ")"

would be a little cleaner since the "SWS" in "LPAREN" is always followed by "LWS" (making it redundant). Using just:

LPAREN = "(" ; case 3
RPAREN = ")"

Would also work since "ctext" allows "LWS". The only problem I see with this is for "( comment one )" the text portion of the comment would be " comment one ". I personally think that allowing leading and trailing white-space around text is a little ugly.

Thanks,

Chris McKinnon

_______________________________________________
Sip-implementors mailing list
[EMAIL PROTECTED]
http://lists.cs.columbia.edu/mailman/listinfo/sip-implementors

Reply via email to