Well, I had not realized that the ONLY use of the L/RPAREN
constructs is in comment. That means we probably should rewrite
comment as:
comment = SWS '(' SWS *(ctext / quoted-pair / comment) SWS ')'
and remove LPAREN and RPAREN productions completely.
You need the inner SWS to get rid of the spaces in your example.
They are whitespace, and they need to be parsed off.
Brian
-----Original Message-----
From: Chris McKinnon [mailto:[EMAIL PROTECTED]
Sent: Monday, August 25, 2003 12:32 PM
To: Mailing List SIP
Subject: RE: [Sip-implementors] Comments
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
