Hi Claus, Thanks for those patches, I appreciate you looking into it. I would prefer though that patches go to me directly instead of posting them on the AG tech list. For example, you supplied a patch for agvcr-rtp-lookup.c but not anything corresponding in rtp.h for the values of RTP_PAYLOAD_MPEG4 etc. I don't want people to read this list, get confused and start patching wildly without understanding what's really going on. So, thanks, but in future can you email me first? Derek Claus Endres wrote:
Hi,
In order to support the MPEG4 and H.264 codecs in the VIC version developed by NCHC in Taiwan, a few simple patches are required to include the RTP Payload Type values for these codecs.
The attached patch is against AGVCR 1.5.8.
There is no disadvantage having these PT values recognized by AGVCR, even if the NCHC extensions are not used.
Regards, Claus.
------------------------------------------------------------------------
--- src/agvcr-rtp-lookup.c.orig 2006-02-23 20:53:19.000000000 +1100 +++ src/agvcr-rtp-lookup.c 2006-02-23 20:55:27.000000000 +1100 @@ -159,11 +159,11 @@ , {RTP_PAYLOAD_UNASSIGNED, "Unassigned", RTP_PAYLOAD_AVTYPE_UNDEFINED} , - {RTP_PAYLOAD_UNASSIGNED, "Unassigned", RTP_PAYLOAD_AVTYPE_UNDEFINED} + {RTP_PAYLOAD_MPEG4, "MPEG4", RTP_PAYLOAD_AVTYPE_VIDEO} , {RTP_PAYLOAD_UNASSIGNED, "Unassigned", RTP_PAYLOAD_AVTYPE_UNDEFINED} , - {RTP_PAYLOAD_UNASSIGNED, "Unassigned", RTP_PAYLOAD_AVTYPE_UNDEFINED} + {RTP_PAYLOAD_H264, "H264", RTP_PAYLOAD_AVTYPE_VIDEO} , {RTP_PAYLOAD_UNASSIGNED, "Unassigned", RTP_PAYLOAD_AVTYPE_UNDEFINED} ,
-- Derek Piper - [email protected] - (812) 856 0111 IRI 323, School of Informatics Indiana University, Bloomington, Indiana