[FFmpeg-trac] #94(avcodec:new): Duplicate #define OPT_STR in libx264.c

FFmpeg trac at avcodec.org
Fri Apr 22 18:56:11 CEST 2011


#94: Duplicate #define OPT_STR in libx264.c
--------------------------+---------------------
 Reporter:  RLeatherwood  |        Type:  defect
   Status:  new           |    Priority:  normal
Component:  avcodec       |     Version:  git
 Keywords:  libx264 x264  |  Blocked By:
 Blocking:                |  Reproduced:  0
 Analyzed:  0             |
--------------------------+---------------------
 After commit 2c18893 to libx264.c, there is a duplicate #define OPT_STR:


 {{{
         if (avctx->bit_rate == 200*100)
             avctx->crf = 23;
     }
 }

 #define OPT_STR(opt,param)
    do {
       if (param && x264_param_parse(&x4->params, opt, param) < 0) { \
           av_log(avctx, AV_LOG_ERROR,                               \
                  "bad value for '%s': '%s'\n", opt, param);         \
           return -1;                                                \
         }                                                           \
    } while (0);                                                     \

 #define OPT_STR(opt,param)
    do {
       if (param && x264_param_parse(&x4->params, opt, param) < 0) { \
           av_log(avctx, AV_LOG_ERROR,                               \
                  "bad value for '%s': '%s'\n", opt, param);         \
           return -1;                                                \
         }                                                           \
    } while (0);                                                     \

 static av_cold int X264_init(AVCodecContext *avctx)
 {
     X264Context *x4 = avctx->priv_data;

 }}}


 I have attached a patch that removes the duplicate #define.

-- 
Ticket URL: <https://ffmpeg.org/trac/ffmpeg/ticket/94>
FFmpeg <http://ffmpeg.org>
FFmpeg issue tracker


More information about the FFmpeg-trac mailing list