On Sat, Feb 8, 2014 at 11:22 PM, Luca Barbato <lu_z...@gentoo.org> wrote:
>
> In short, do not use av_assert* in new code you are committing to the
> mainline.
>

What should I do about the assert in av_clipd_c?

/**
 * Clip a double value into the amin-amax range.
 * @param  a     value to clip
 * @param  amin  minimum value of the clip range
 * @param  amax  maximum value of the clip range
 * @return clipped value
 */
static av_always_inline av_const double av_clipd_c(double a, double
amin, double amax)
{
    av_assert2(amin <= amax);
    if (a < amin)
        return amin;
    else if (a > amax)
        return amax;
    else
        return a;
}

Should I simply remove it?
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to