diff --git a/libpostproc/postprocess.c b/libpostproc/postprocess.c index 01ec0f9867..6dee0417e5 100644 --- a/libpostproc/postprocess.c +++ b/libpostproc/postprocess.c @@ -209,13 +209,13 @@ static inline int isHorizDC_C(const uint8_t src[], int stride, const PPContext * const int dcThreshold= dcOffset*2 + 1; for(y=0; y c->ppMode.flatnessThreshold; @@ -233,14 +233,14 @@ static inline int isVertDC_C(const uint8_t src[], int stride, const PPContext *c src+= stride*4; // src points to begin of the 8x8 Block for(y=0; y c->ppMode.flatnessThreshold; @@ -278,10 +278,7 @@ static inline int isVertMinMaxOk_C(const uint8_t src[], int stride, int QP) static inline int horizClassify_C(const uint8_t src[], int stride, const PPContext *c) { if( isHorizDC_C(src, stride, c) ){ - if( isHorizMinMaxOk_C(src, stride, c->QP) ) - return 1; - else - return 0; + return isHorizMinMaxOk_C(src, stride, c->QP); }else{ return 2; } @@ -290,10 +287,7 @@ static inline int horizClassify_C(const uint8_t src[], int stride, const PPConte static inline int vertClassify_C(const uint8_t src[], int stride, const PPContext *c) { if( isVertDC_C(src, stride, c) ){ - if( isVertMinMaxOk_C(src, stride, c->QP) ) - return 1; - else - return 0; + return isVertMinMaxOk_C(src, stride, c->QP); }else{ return 2; } @@ -704,21 +698,22 @@ pp_mode *pp_get_mode_by_name_and_quality(const char *name, int quality) av_log(NULL, AV_LOG_DEBUG, "pp: %s\n", name); for(;;){ - char *filterName; + const char *filterName; int q= 1000000; //PP_QUALITY_MAX; int chrom=-1; int luma=-1; - char *option; - char *options[OPTIONS_ARRAY_SIZE]; + const char *option; + const char *options[OPTIONS_ARRAY_SIZE]; int i; int filterNameOk=0; int numOfUnknownOptions=0; int enable=1; //does the user want us to enabled or disabled the filter + char *tokstate; - filterToken= strtok(p, filterDelimiters); + filterToken= av_strtok(p, filterDelimiters, &tokstate); if(!filterToken) break; p+= strlen(filterToken) + 1; // p points to next filterToken - filterName= strtok(filterToken, optionDelimiters); + filterName= av_strtok(filterToken, optionDelimiters, &tokstate); if (!filterName) { ppMode->error++; break; @@ -731,7 +726,7 @@ pp_mode *pp_get_mode_by_name_and_quality(const char *name, int quality) } for(;;){ //for all options - option= strtok(NULL, optionDelimiters); + option= av_strtok(NULL, optionDelimiters, &tokstate); if(!option) break; av_log(NULL, AV_LOG_DEBUG, "pp: option: %s\n", option); diff --git a/tests/fate/filter-video.mak b/tests/fate/filter-video.mak index 9239b379dd..c49cf00388 100644 --- a/tests/fate/filter-video.mak +++ b/tests/fate/filter-video.mak @@ -229,6 +229,9 @@ fate-filter-pad: CMD = video_filter "pad=iw*1.5:ih*1.5:iw*0.3:ih*0.2" FATE_FILTER_VSYNTH-$(CONFIG_PP_FILTER) += fate-filter-pp fate-filter-pp: CMD = video_filter "pp=be/hb/vb/tn/l5/al" +FATE_FILTER_VSYNTH-$(CONFIG_PP_FILTER) += fate-filter-pp1 +fate-filter-pp1: CMD = video_filter "pp=fq|4/be/hb/vb/tn/l5/al" + FATE_FILTER_VSYNTH-$(CONFIG_PP_FILTER) += fate-filter-pp2 fate-filter-pp2: CMD = video_filter "pp=be/fq|16/h1/v1/lb" diff --git a/tests/ref/fate/filter-pp1 b/tests/ref/fate/filter-pp1 new file mode 100644 index 0000000000..b129ea39ee --- /dev/null +++ b/tests/ref/fate/filter-pp1 @@ -0,0 +1 @@ +pp1 cb9f884e27a5be11f72afc9b517efd10