[AG-TECH] AGVCR support of MPEG4 and H.264

[Derek Piper]

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