[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[MMUSIC] Comments on draft-ietf-mmusic-sdp-srcfilter-01.txt
- To: IETF MMUSIC WG <mmusic@ietf.org>
- Subject: [MMUSIC] Comments on draft-ietf-mmusic-sdp-srcfilter-01.txt
- From: Magnus Westerlund <magnus.westerlund@era.ericsson.se>
- Date: Mon, 08 Jul 2002 17:04:19 +0200
- List-Id: Multiparty Multimedia Session Control Working Group <mmusic.ietf.org>
- Sender: mmusic-admin@ietf.org
- User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.4) Gecko/20011128 Netscape6/6.2.1
Hi,
I have some comments about the draft.
Section 2 and Appendix A.
The definition of the filter attribute. Wouldn't it be better to define
a short attribute name to the source filter instead of having two
attributes called incl and excl? I would propose
a=SF:<filter-mode>:<filter-spec>.
Section 2.2.
The media description 1 port number is not even, Please change to 54320
which is a possible number. Also the payload type for video shoul be
changed. A possible value would be 34 that are a static payload type for
h.263 video.
Section 2.2.4
The example text is in contradiction with the processing rules in
section 2.1. According to the processing rules all three src adresses
(1.1.1.1, 2.2.2.2 and 3.3.3.3) are valid for any of the three multicast
adresses.
Appendix A.
The definition of src-list i believe is in error. The addresses should
not be seperate by email-safe but rather SP.
The definition from draft-ietf-mmusic-sdp-new-10.txt says:
email-safe = 1*(%x01-09/%x0B-0C/%x0E-27/
%x2A-3B/%x3D/%x3E-FF)
;any byte except NUL, CR, LF, or the quoting
;characters ()<>
Which includes lots of more charachters than space which is referenced
in the text in section 2.
Regards
Magnus Westerlund
Multimedia Technologies, Ericsson Research ERA/TVA/A
----------------------------------------------------------------------
Ericsson AB | Phone +46 8 4048287
Torshamsgatan 23 | Fax +46 8 7575550
S-164 80 Stockholm, Sweden | mailto: magnus.westerlund@era.ericsson.se
_______________________________________________
mmusic mailing list
mmusic@ietf.org
https://www1.ietf.org/mailman/listinfo/mmusic