[FFmpeg-devel] [PATCH] als in mp4

Jai Menon jmenon86
Fri Apr 17 18:52:35 CEST 2009


On 4/9/09, Jai Menon <jmenon86 at gmail.com> wrote:
> On 4/9/09, Baptiste Coudurier <baptiste.coudurier at gmail.com> wrote:
>  > Hi Jai,
>  >
>  >
>  >  On 4/9/2009 12:06 AM, Jai Menon wrote:
>  >  > On Wed, Apr 8, 2009 at 10:38 AM, Baptiste Coudurier
>  >  > <baptiste.coudurier at gmail.com> wrote:
>  >  >> Hi Jai,
>  >  >>
>  >  >> On 4/6/2009 5:12 AM, Jai Menon wrote:
>  >  >>> On Fri, Apr 3, 2009 at 11:28 PM, Baptiste Coudurier

[...]

> So how should we do this in your opinion? There should be some way we
>  can instruct the demuxer that chan_config is an absolute value and not
>  an index. The patch does this using the absolute_channels flag. As for
>  the bits_per_sample field, I will remove it in the next patch.

Okay, I'm attaching the bare minimum patchset required. This should be
pretty uncontroversial. I have no idea how to simplify this further.
Decoder will be required for setting channels. I haven't attached the
patch for muxing (which is another hack in itself). Is this fine?

-- 
Regards,

Jai
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-add-mp4-als-codec-id-and-bump-minor.patch
Type: text/x-patch
Size: 906 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090417/964edb3c/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-hook-als-aot-to-it-lavc-codec-id.patch
Type: text/x-patch
Size: 734 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090417/964edb3c/attachment-0001.bin>



More information about the ffmpeg-devel mailing list