[Asterisk-video] app_mp4 patch

[Sergio Garcia]

/* What for is this ? */
//	if (payload > 0)
//		MP4AddRtpImmediateData(t->mp4, t->hint, f->data, payload);


The inmediate data is the part of the rtp data that is specific to the rtp payload. So when you receive a frame of a custom media you have to decide which payload are you going to save in the rtp data (the hint track) and which part in the media track and then set the reference in the hint track.
Yuo can also save all the media track in the hint track (if you've ever used the qt pro that's what the option "optimize for server does").

If you discard the payload you'll lose the first byte when trying to build the rtp packet again.


audio = MP4AddAmrAudioTrack(mp4, 8000, 0, 0, 1, 0); /* Should check framesPerSample*/

You are rigth, but currently only 1 frame per sample is supported, and probably would need a bit work to support other calues.


/* may this cause problems as it is hardcoded to 98 */
type = 98;

No, it shouldn't, you can choose whatever dinamyc payload you like.

Could you send a patch file with the diffs?? Also, if you want I could set up an svn account so you could patch the code directly.

Francesco, could you also patch the bitreverse bits stuff with your latest version before??


Best regards
Sergio






---------- Original Message ----------------------------------
From: Klaus Darilion <klaus.mailinglists at pernau.at>
Reply-To: Development discussion of video media support in Asterisk<asterisk-video at lists.digium.com>
Date:  Mon, 30 Jul 2007 12:57:47 +0200

>regards
>klaus
>
>
>