For generality, qsort uses a comparator whose elements are void *. This makes the comparator have such a form, and thus makes the void * cast of the comparator pointer useless. Furthermore, this makes the code more consistent with other usages of qsort across the codebase.
Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> --- libavfilter/vf_deshake.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavfilter/vf_deshake.c b/libavfilter/vf_deshake.c index ac13ecd..3f96f7d 100644 --- a/libavfilter/vf_deshake.c +++ b/libavfilter/vf_deshake.c @@ -91,9 +91,10 @@ static const AVOption deshake_options[] = { AVFILTER_DEFINE_CLASS(deshake); -static int cmp(const double *a, const double *b) +static int cmp(const void *a, const void *b) { - return *a < *b ? -1 : ( *a > *b ? 1 : 0 ); + double va = *(double *)a, vb = *(double *)b; + return va < vb ? -1 : ( va > vb ? 1 : 0 ); } /** @@ -105,7 +106,7 @@ static double clean_mean(double *values, int count) int cut = count / 5; int x; - qsort(values, count, sizeof(double), (void*)cmp); + qsort(values, count, sizeof(double), cmp); for (x = cut; x < count - cut; x++) { mean += values[x]; -- 2.6.2 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel