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 - dcpi...@indiana.edu - (812) 856 0111
IRI 323, School of Informatics
Indiana University, Bloomington, Indiana
