[ABRT PATCH] abrt-*-client: simplify formatting of locale-related headers

Michal Toman mtoman at redhat.com
Thu Sep 12 12:31:50 UTC 2013


Pushed with one minor change

On 09.09.2013 14:31, Denys Vlasenko  wrote:
> @@ -382,21 +382,14 @@ static int check_package(const char *nvr, const char *arch, map_string_t *osinfo
>                          "X-Package-NVR: %s\r\n"
>                          "X-Package-Arch: %s\r\n"
>                          "X-OS-Release: %s\r\n",
                    removed the excessive comma   ^
> -                       cfg.url, nvr, arch, releaseid);
> -
> -    if (lang.encoding)
> -        strbuf_append_strf(http_request,
> -                           "Accept-Charset: %s\r\n",
> -                           lang.encoding);
> -    if (lang.locale)
> -    {
> -        strbuf_append_strf(http_request,
> -                           "Accept-Language: %s\r\n",
> -                           lang.locale);
> -        free(lang.locale);
> -    }
> +                       "%s"
> +                       "%s"
> +                       "\r\n",
> +                       cfg.url, nvr, arch, releaseid,
> +                       lang.accept_charset,
> +                       lang.accept_language
> +    );


More information about the Crash-catcher mailing list