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

[Michal Toman]

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
+    );