[Asterisk-video] app_mp4 patch

[Atle Samuelsen]

Hi Sergio,

This patch atleast works somewhat bether then the original version in
svn :) 

I have´nt had any coredumps with thisone :) as I had with the one in
svn :)

Only problem (wich I think is also related to the hint stuff) is that >I
cant get the video plaied off.. and get some intresting errors.

- Atle

* Sergio Garcia <sergio.garcia at fontventa.com> [070730 15:25]:
> 
> /* 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
> >
> >
> >
>  
> 
> _______________________________________________
> --Bandwidth and Colocation Provided by http://www.api-digital.com--
> 
> asterisk-video mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-video