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