[FFmpeg-devel] [PATCH 1/2] lavf/matroska: Add functions for WebM DASH Manifest

[wm4]

On Mon, 7 Jul 2014 12:46:36 -0700
Vignesh Venkatasubramanian <vigneshv at google.com> wrote:

> On Mon, Jul 7, 2014 at 12:30 PM, wm4 <nfxjfg at googlemail.com> wrote:
> > On Mon, 7 Jul 2014 11:53:40 -0700
> > Vignesh Venkatasubramanian <vigneshv at google.com> wrote:
> >
> >> On Mon, Jul 7, 2014 at 11:32 AM, wm4 <nfxjfg at googlemail.com> wrote:
> >> > On Mon, 7 Jul 2014 09:27:22 -0700
> >> > Vignesh Venkatasubramanian <vigneshv at google.com> wrote:
> >> >
> >> >> >> +static int webm_dash_manifest_read_packet(AVFormatContext *s, AVPacket *pkt)
> >> >> >> +{
> >> >> >> +    return AVERROR_EOF;
> >> >> >> +}
> >> >> >
> >> >> > Shouldn't this return packets? Maybe I'm misunderstanding the purpose
> >> >> > of this code.
> >> >> >
> >> >>
> >> >> the point of this demuxer is to read certain metadata and pass it
> >> >> along to the muxer so that it can generate the manifest XML. for this
> >> >> purpose, it does not have to read any packets from the file at all,
> >> >> just the headers will do. so we don't return any packets.
> >> >
> >> > I see... well, I hope there will be a ffmpeg native demuxer for this
> >> > too? Maybe it could be made a condition for merging the muxer.
> >>
> >> i'm not sure what you mean by a ffmpeg native demuxer. can you please
> >> elaborate a little more?
> >
> > A demuxer that can read the DASH manifest produced by ffmpeg and
> > accompanying files.
> 
> ah, alright. i'm not sure how much use would that be. anyway, for now
> i'm adding only the muxer.

Lots of use not only for testing (it would be very odd if you couldn't
feed back a ffmpeg-generated file to ffmpeg or tools like ffprobe), but
also for any software that uses libavformat.