[FFmpeg-devel] [PATCH]lavf/amr: Add amrnb and amrwb demuxers

[wm4]

On Wed, 27 Sep 2017 18:47:16 +0200
Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2017-09-27 18:40 GMT+02:00 wm4 <nfxjfg at googlemail.com>:
> > On Wed, 27 Sep 2017 18:08:19 +0200
> > Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> >  
> >> Hi!
> >>
> >> The existing amr demuxer does not allow reading streams, it requires
> >> the 3GPP-conforming file header.
> >> Attached patch allows reading amrnb and amrwb from (live) streams,
> >> fixes ticket #6678.
> >>
> >> Please comment, Carl Eugen  
> >
> > That seems particularly pointless, since the user has to force the
> > demuxers.  
> 
> Like for any other demuxer without auto-detection?

Actually no.

> I wasn't able to "force" reading such a stream without this patch,
> am I missing something?

Then maybe there should be a mechanism to force stream parameters, just
like they obviously exist for raw audio/video.

> > I don't know what that ticket is  
> 
> Then it may be better not to comment.

It may be better for you not committing anything if you don't know what
commit messages are.