[FFmpeg-devel] [PATCH 2/2] ffprobe: Rename variable in csv_escape_str()

[Stefano Sabatini]

On date Tuesday 2012-09-18 23:11:28 +0200, Alexander Strasser encoded:
> Sync to 3245be9 .
> 
> Signed-off-by: Alexander Strasser <eclipse7 at gmx.net>
> ---
>  ffprobe.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/ffprobe.c b/ffprobe.c
> index a226d0d..aad5d90 100644
> --- a/ffprobe.c
> +++ b/ffprobe.c
> @@ -566,9 +566,9 @@ static const char *c_escape_str(AVBPrint *dst, const char *src, const char sep,
>   */
>  static const char *csv_escape_str(AVBPrint *dst, const char *src, const char sep, void *log_ctx)
>  {
> -    int quote = !!src[strcspn(src, "\",\n\r")];
> +    int needs_quoting = !!src[strcspn(src, "\",\n\r")];
>  
> -    if (quote)
> +    if (needs_quoting)
>          av_bprint_chars(dst, '\"', 1);
>  
>      for (; *src; src++) {
> @@ -576,7 +576,7 @@ static const char *csv_escape_str(AVBPrint *dst, const char *src, const char sep
>              av_bprint_chars(dst, '\"', 1);
>          av_bprint_chars(dst, *src, 1);
>      }
> -    if (quote)
> +    if (needs_quoting)
>          av_bprint_chars(dst, '\"', 1);
>      return dst->str;
>  }

LGTM.
-- 
FFmpeg = Furious Fast Multimedia Patchable Elastic Gladiator