[FFmpeg-devel] [PATCH]lavf/amr: Add amrnb and amrwb demuxers
nfxjfg at googlemail.com
Wed Sep 27 19:58:09 EEST 2017
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?
> 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.
More information about the ffmpeg-devel