[FFmpeg-devel] [PATCH 1/5] avformat/ffmdec: Remove some st->codec uses which set encoder parameters

Michael Niedermayer michael at niedermayer.cc
Mon Dec 5 02:27:31 EET 2016


On Mon, Dec 05, 2016 at 12:28:49AM +0100, Moritz Barsnick wrote:
> On Sat, Dec 03, 2016 at 13:23:26 +0100, Michael Niedermayer wrote:
> > > Please add a comment like /* gop_size */ here and similar ones below.
> > > Otherwise it'll be rather mysterious what's the point of all these avio_rb* calls.
> > 
> > done locally
> 
> Are the remaining avio_rb*() calls good for anything except documenting
> the now unused fields? I know the code is just called once for the

Their only use is probably documenting


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161205/e43d9711/attachment.sig>


More information about the ffmpeg-devel mailing list