@b4n commented on this pull request.

		/* retval before function */

g_string_prepend_c(str, ' '); g_string_prepend(str, retval); break; } + if (sep) + { + g_string_append(str, sep); + g_string_append(str, retval); + }

I'd rather have an if/else here if the value of `sep` is the criteria, like that I find the code weird as well. ```suggestion break; }

if (sep) { /* retval after function */ g_string_append(str, sep); g_string_append(str, retval); } else { /* retval before function */ g_string_prepend_c(str, ' '); g_string_prepend(str, retval); } ```

...

Hmmm, yes the original didn't make sense did it?

it was just a bit of a bizarre way of laying out the code.

See edit adding the words I thought would be understood. :grin:

Merged #3463 into master.