Hi, On Sat, Jul 2, 2011 at 12:09 AM, Anton Khirnov <[email protected]> wrote: > On Tue, 21 Jun 2011 09:20:37 -0400, "Ronald S. Bultje" <[email protected]> > wrote: >> On Tue, Jun 21, 2011 at 2:44 AM, Anton Khirnov <[email protected]> wrote: >> > AVStream is no place for it and it's unused outside of ffmpeg anyway. >> > --- >> > ffmpeg.c | 15 ++++++++------- >> > libavformat/avformat.h | 4 +++- >> > libavformat/ffmdec.c | 1 - >> > libavformat/ffmenc.c | 1 - >> > libavformat/version.h | 3 +++ >> > tests/ref/lavf/ffm | 2 +- >> > 6 files changed, 15 insertions(+), 11 deletions(-) >> > --- a/libavformat/ffmenc.c >> > +++ b/libavformat/ffmenc.c >> > @@ -113,7 +113,6 @@ static int ffm_write_header(AVFormatContext *s) >> > avio_wb32(pb, codec->codec_id); >> > avio_w8(pb, codec->codec_type); >> > avio_wb32(pb, codec->bit_rate); >> > - avio_wb32(pb, st->quality); >> > avio_wb32(pb, codec->flags); >> > avio_wb32(pb, codec->flags2); >> > avio_wb32(pb, codec->debug); >> > --- a/tests/ref/lavf/ffm >> > +++ b/tests/ref/lavf/ffm >> > @@ -1,3 +1,3 @@ >> > -b6acf782a38d313153b68c4ca204fc90 *./tests/data/lavf/lavf.ffm >> > +f9bee27ea1b6b83a06b5f9efb0a4ac1f *./tests/data/lavf/lavf.ffm >> > 376832 ./tests/data/lavf/lavf.ffm >> > ./tests/data/lavf/lavf.ffm CRC=0xf361ed74 >> >> Can we store this in metadata somehow, so we can still transmit it >> between ffmpeg and ffserver? > > Why would we want to? ffserver doesn't use this field. > Nothing except ffmpeg uses this field.
Yeah I guess it's fine - just push it. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
