[FFmpeg-devel] [PATCH] libnut: Pass stream to ff_parse_specific_params

Derek Buitenhuis derek.buitenhuis at gmail.com
Thu Apr 21 14:52:46 CEST 2016


This function bo longer takes an AVCodecContext.

Fixes ticket #5430.

Signed-off-by: Derek Buitenhuis <derekb at vimeo.com>
---
Since I'm sending this patch anyway, I had a few questions:

  1. Where the heck is libnut's official repo. Google only turns up
     random GitHub forks, and a link to MultiMediawiki, which in turn
     has a link to git://git.ffmpeg.org/nut, which is not listed on ANY
     website except this wiki, and, which in turn, is a git mirror of
     svn://svn.mplayerhq.hu/nut, which I also cannot find listed anywhere.
     As far as I can tell, libnut does not exist in any searchable way.
  2. Given that we have both a native nut muxer and demuxer, and what I
     listed in the point above, why do we keep libnut.c? It took *two
     weeks* for someone to even notice it didn't compiled, and even
     they didn't give any legitimate reason why they needed it.

Food for thought.
---
 libavformat/libnut.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/libnut.c b/libavformat/libnut.c
index 48f0928..26bb354 100644
--- a/libavformat/libnut.c
+++ b/libavformat/libnut.c
@@ -94,7 +94,7 @@ static int nut_write_header(AVFormatContext * avf) {
         s[i].fourcc = av_malloc(s[i].fourcc_len);
         for (j = 0; j < s[i].fourcc_len; j++) s[i].fourcc[j] = (fourcc >> (j*8)) & 0xFF;
 
-        ff_parse_specific_params(codec, &num, &ssize, &denom);
+        ff_parse_specific_params(avf->streams[i], &num, &ssize, &denom);
         avpriv_set_pts_info(avf->streams[i], 60, denom, num);
 
         s[i].time_base.num = denom;
-- 
1.8.3.1



More information about the ffmpeg-devel mailing list