[FFmpeg-devel] [PATCH] lavu/common: add warning to GET_UTF8 doxy

Stefano Sabatini stefasab at gmail.com
Fri Oct 4 10:46:56 CEST 2013


On date Thursday 2013-10-03 00:34:03 +0200, Stefano Sabatini encoded:
> Should prevent wrong uses, or at least decrease their chance.
> ---
>  libavutil/common.h | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/libavutil/common.h b/libavutil/common.h
> index 9f0f998..b1203ad 100644
> --- a/libavutil/common.h
> +++ b/libavutil/common.h
> @@ -295,6 +295,11 @@ static av_always_inline av_const int av_popcount64_c(uint64_t x)
>   *                 input, this could be *ptr++.
>   * @param ERROR    Expression to be evaluated on invalid input,
>   *                 typically a goto statement.
> + *
> + * @warning ERROR should not contain a loop control statement which
> + * could interact with the internal while loop, and should force an
> + * exit from the macro code (e.g. through a goto or a return) in order
> + * to prevent undefined results.
>   */
>  #define GET_UTF8(val, GET_BYTE, ERROR)\
>      val= GET_BYTE;\

Ping.
-- 
FFmpeg = Fanciful and Faithful Mysterious Patchable Elegant Geek


More information about the ffmpeg-devel mailing list