Code review comment for lp:~charlesk/indicator-sound/blacklist-leaks

Revision history for this message
Charles Kerr (charlesk) wrote :

g_variant_get("(s)") gets a newly-allocated string
g_variant_get("(&s)") peeks the variant's internal pointer to be treated as const

We were doing the former and leaking the string; this patch uses the latter and adds const

« Back to merge proposal