[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: appstream-glib: FTBFS on hurd-i396 (for review)



On 22/10/14 12:01, Samuel Thibault wrote:
Svante Signell, le Wed 22 Oct 2014 11:33:39 +0200, a écrit :
On Wed, 2014-10-22 at 11:11 +0200, Pino Toscano wrote:
On 2014-10-22 10:59, Svante Signell wrote:


Updated patch attached. For the first part of the patch, I don't find
using g_strdup_printf() is an advantage over g_malloc(), since later
on
g_realloc() is needed.

The other option is to just use a different buffer for each path, which
could actually simplify the code (you are also not checking whether a
new length is smaller than the previous one, to avoid realloc).

Yes, that could be a solution too, Samuel?

What looks nicer is usually what maintainers prefer.

That would most likely be g_strdup_printf() and g_free(). Please use that.

Emilio


Reply to: