postproc: Avoid floats in maxClipped computation
authorMichael Niedermayer <michaelni@gmx.at>
Sat, 30 May 2015 23:40:12 +0000 (01:40 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Sat, 30 May 2015 23:40:12 +0000 (01:40 +0200)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libpostproc/postprocess.c
libpostproc/postprocess_internal.h
libpostproc/postprocess_template.c

index acce0f2..92cc436 100644 (file)
@@ -681,7 +681,7 @@ pp_mode *pp_get_mode_by_name_and_quality(const char *name, int quality)
     ppMode->minAllowedY= 16;
     ppMode->baseDcDiff= 256/8;
     ppMode->flatnessThreshold= 56-16-1;
-    ppMode->maxClippedThreshold= 0.01;
+    ppMode->maxClippedThreshold= (AVRational){1,100};
     ppMode->error=0;
 
     memset(temp, 0, GET_MODE_BUFFER_SIZE);
index c1a306d..d128dfb 100644 (file)
@@ -99,7 +99,7 @@ typedef struct PPMode{
 
     int minAllowedY;                ///< for brightness correction
     int maxAllowedY;                ///< for brightness correction
-    float maxClippedThreshold;      ///< amount of "black" you are willing to lose to get a brightness-corrected picture
+    AVRational maxClippedThreshold; ///< amount of "black" you are willing to lose to get a brightness-corrected picture
 
     int maxTmpNoise[3];             ///< for Temporal Noise Reducing filter (Maximal sum of abs differences)
 
index fc49e19..c7ad3f0 100644 (file)
@@ -3383,7 +3383,7 @@ static void RENAME(postProcess)(const uint8_t src[], int srcStride, uint8_t dst[
         }
 
         /* We always get a completely black picture first. */
-        maxClipped= (uint64_t)(sum * c.ppMode.maxClippedThreshold);
+        maxClipped= av_rescale(sum, c.ppMode.maxClippedThreshold.num, c.ppMode.maxClippedThreshold.den);
 
         clipped= sum;
         for(black=255; black>0; black--){