[FFmpeg-devel] [PATCH 3/4] lavf/segment: Rename variable in print_csv_escaped_str()

Stefano Sabatini stefasab at gmail.com
Sat Sep 15 10:09:15 CEST 2012


On date Saturday 2012-09-15 01:07:56 +0200, Alexander Strasser encoded:
> Rename a local variable to make the code more self-explanatory.
> 
> Signed-off-by: Alexander Strasser <eclipse7 at gmx.net>
> ---
>  libavformat/segment.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/libavformat/segment.c b/libavformat/segment.c
> index 07b8da3..fbfd677 100644
> --- a/libavformat/segment.c
> +++ b/libavformat/segment.c
> @@ -76,9 +76,9 @@ typedef struct {
>  
>  static void print_csv_escaped_str(AVIOContext *ctx, const char *str)
>  {
> -    int quote = !!str[strcspn(str, "\",\n\r")];
> +    int needs_quoting = !!str[strcspn(str, "\",\n\r")];
>  
> -    if (quote)
> +    if (needs_quoting)
>          avio_w8(ctx, '"');
>  
>      for (; *str; str++) {
> @@ -86,7 +86,7 @@ static void print_csv_escaped_str(AVIOContext *ctx, const char *str)
>              avio_w8(ctx, '"');
>          avio_w8(ctx, *str);
>      }
> -    if (quote)
> +    if (needs_quoting)
>          avio_w8(ctx, '"');
>  }

LGTM if you prefer like that..
-- 
FFmpeg = Fierce and Formidable MultiPurpose Extroverse Game


More information about the ffmpeg-devel mailing list