From d5ec8ba7f2cb83a166da196c377398b7954c5d6c Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Sun, 15 Sep 2013 15:25:09 +0200 Subject: Do not leave positive values undefined when negative are defined as error Define positive return values as non errors and leave further meaning undefined This allows future extensions to use these values Reviewed-by: Paul B Mahol Signed-off-by: Michael Niedermayer --- libavfilter/internal.h | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'libavfilter/internal.h') diff --git a/libavfilter/internal.h b/libavfilter/internal.h index 5ea3cad..f8d0cce 100644 --- a/libavfilter/internal.h +++ b/libavfilter/internal.h @@ -169,7 +169,7 @@ int ff_fmt_is_in(int fmt, const int *fmts); * @param ret pixel format pointer to where the value should be written * @param arg string to parse * @param log_ctx log context - * @return 0 in case of success, a negative AVERROR code on error + * @return >= 0 in case of success, a negative AVERROR code on error */ int ff_parse_pixel_format(enum AVPixelFormat *ret, const char *arg, void *log_ctx); @@ -179,7 +179,7 @@ int ff_parse_pixel_format(enum AVPixelFormat *ret, const char *arg, void *log_ct * @param ret unsigned integer pointer to where the value should be written * @param arg string to parse * @param log_ctx log context - * @return 0 in case of success, a negative AVERROR code on error + * @return >= 0 in case of success, a negative AVERROR code on error */ int ff_parse_sample_rate(int *ret, const char *arg, void *log_ctx); @@ -189,7 +189,7 @@ int ff_parse_sample_rate(int *ret, const char *arg, void *log_ctx); * @param ret unsigned AVRational pointer to where the value should be written * @param arg string to parse * @param log_ctx log context - * @return 0 in case of success, a negative AVERROR code on error + * @return >= 0 in case of success, a negative AVERROR code on error */ int ff_parse_time_base(AVRational *ret, const char *arg, void *log_ctx); @@ -199,7 +199,7 @@ int ff_parse_time_base(AVRational *ret, const char *arg, void *log_ctx); * @param ret integer pointer to where the value should be written * @param arg string to parse * @param log_ctx log context - * @return 0 in case of success, a negative AVERROR code on error + * @return >= 0 in case of success, a negative AVERROR code on error */ int ff_parse_sample_format(int *ret, const char *arg, void *log_ctx); @@ -209,7 +209,7 @@ int ff_parse_sample_format(int *ret, const char *arg, void *log_ctx); * @param ret 64bit integer pointer to where the value should be written. * @param arg string to parse * @param log_ctx log context - * @return 0 in case of success, a negative AVERROR code on error + * @return >= 0 in case of success, a negative AVERROR code on error */ int ff_parse_channel_layout(int64_t *ret, const char *arg, void *log_ctx); -- cgit v1.1