On Mon, Jan 08, 2018 at 10:56:56 +0530, vdi...@akamai.com wrote:

> -    { "audio_pts",     "audio pts source",   OFFSET(audio_pts_source),    
> AV_OPT_TYPE_INT,   { .i64 = PTS_SRC_AUDIO    }, 1, 4, DEC, "pts_source"},
> -    { "video_pts",     "video pts source",   OFFSET(video_pts_source),    
> AV_OPT_TYPE_INT,   { .i64 = PTS_SRC_VIDEO    }, 1, 4, DEC, "pts_source"},
> +    { "audio_pts",     "audio pts source",   OFFSET(audio_pts_source),    
> AV_OPT_TYPE_INT,   { .i64 = PTS_SRC_AUDIO    }, 1, 5, DEC, "pts_source"},
> +    { "video_pts",     "video pts source",   OFFSET(video_pts_source),    
> AV_OPT_TYPE_INT,   { .i64 = PTS_SRC_VIDEO    }, 1, 5, DEC, "pts_source"},

This cries for a max macro:

     PTS_SRC_VIDEO     = 2,
     PTS_SRC_REFERENCE = 3,
     PTS_SRC_WALLCLOCK = 4,
+    PTS_SRC_ABS_WALLCLOCK = 5,
+    PTS_SRC_NB
 } DecklinkPtsSource;

and then

+    { "audio_pts",     "audio pts source",   OFFSET(audio_pts_source),    
AV_OPT_TYPE_INT,   { .i64 = PTS_SRC_AUDIO    }, 1, PTS_SRC_NB-1, DEC, 
"pts_source"},
+    { "video_pts",     "video pts source",   OFFSET(video_pts_source),    
AV_OPT_TYPE_INT,   { .i64 = PTS_SRC_VIDEO    }, 1, PTS_SRC_NB-1, DEC, 
"pts_source"},

>              break;
>          case PTS_SRC_WALLCLOCK:
> +        case PTS_SRC_ABS_WALLCLOCK:

Coverty and similar tools like fall-throughs to be marked as such to
avoid false warnings.

> +    else if(ctx->audio_pts_source == PTS_SRC_ABS_WALLCLOCK || 
> ctx->video_pts_source == PTS_SRC_ABS_WALLCLOCK)

"else if ("

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to