On Mon, Sep 19, 2016 at 08:54:40PM +0200, Kevin Daudt wrote:

> diff --git a/mailinfo.c b/mailinfo.c
> index e19abe3..6a7c2f2 100644
> --- a/mailinfo.c
> +++ b/mailinfo.c
> @@ -54,6 +54,50 @@ static void parse_bogus_from(struct mailinfo *mi, const 
> struct strbuf *line)
>       get_sane_name(&mi->name, &mi->name, &mi->email);
>  }
>  
> +static void unquote_quoted_string(struct strbuf *line)
> +{
> +     const char *in = strbuf_detach(line, NULL);

I see that this version uses the "detach, and then write into the
replacement" approach, which is good. But...

> +     int c, take_next_literally = 0;
> +     int found_error = 0;
> +
> +     /*
> +      * Stores the character that started the escape mode so that we know 
> what
> +      * character will stop it
> +      */
> +     char escape_context = 0;
> +
> +     while ((c = *in++) != 0) {
> +             if (take_next_literally) {
> +                     take_next_literally = 0;
> +             } else {
> [...]
> +             }
> +
> +             strbuf_addch(line, c);
> +     }
> +}

It needs to `free(in)` at the end of the function.

Your original also fed "line->len" as a hint, but I doubt it really
matters in practice, so I don't mind losing that.

-Peff

Reply via email to