Re: [FFmpeg-devel] [PATCH]lavf/mpeg: Identify sub-stream ID 0xa1 as mlp

2017-03-02 Thread Hendrik Leppkes 
On Wed, Mar 9, 2016 at 2:14 PM, Carl Eugen Hoyos  wrote:
> Michael Niedermayer  niedermayer.cc> writes:
>
>> >  mpeg.c |2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> > 441c00cda76c5f59b560517115ad1677ab8061bd  patchmpegmlp.diff
>> > diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
>> > index 787d5f0..2f48459 100644
>>
>> if that works and no better way is known then it should be ok
>
> Patch applied.
>

A user shared a private sample with me that uses 0xa1 for a lpcm
stream, which gets mis-detected as mlp now.
It appears this would not be quite the final solution. AOB may use
different mappings then ordinary VOB/MPEG files, is there no way to
differentiate the different kinds?

- Hendrik
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavf/mpeg: Identify sub-stream ID 0xa1 as mlp

2016-03-09 Thread Carl Eugen Hoyos 
Michael Niedermayer  niedermayer.cc> writes:

> >  mpeg.c |2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 441c00cda76c5f59b560517115ad1677ab8061bd  patchmpegmlp.diff
> > diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
> > index 787d5f0..2f48459 100644
> 
> if that works and no better way is known then it should be ok

Patch applied.

Thank you, Carl Eugen

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavf/mpeg: Identify sub-stream ID 0xa1 as mlp

2016-03-03 Thread Michael Niedermayer 
On Thu, Mar 03, 2016 at 11:26:20AM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch (still) fixes ticket #4786 for me.
> Autodetection seems impossible to me and the following 
> page suggests that this patch is likely correct:
> http://dvd-audio.sourceforge.net/spec/aob.shtml
> 
> Please comment, Carl Eugen

>  mpeg.c |2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 441c00cda76c5f59b560517115ad1677ab8061bd  patchmpegmlp.diff
> diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
> index 787d5f0..2f48459 100644

if that works and no better way is known then it should be ok

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel