syntax-check: add syntactic rule to prohibit gs_strfreev

and fix an instance.

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
This commit is contained in:
Giuseppe Scrivano 2015-05-12 09:41:44 +02:00
parent f38ec6dfdf
commit e3a3e2b618
2 changed files with 6 additions and 1 deletions

View File

@ -1032,6 +1032,11 @@ sc_const_long_option:
halt='add "const" to the above declarations' \
$(_sc_search_regexp)
sc_prohibit_gs_strfreev:
@prohibit='\bgs_strfreev\b' \
halt="do not use gs_strfreev; use g_autoptr instead" \
$(_sc_search_regexp)
sc_prohibit_gs_unref:
@prohibit='\bgs_unref_(keyfile|variant*|ptrarray|hashtable|bytes|object)\b' \
halt="do not use gs_unref_*; use g_autoptr instead" \

View File

@ -1501,7 +1501,7 @@ _ostree_parse_delta_name (const char *delta_name,
char **out_from,
char **out_to)
{
gs_strfreev char **parts = g_strsplit (delta_name, "-", 2);
g_auto(GStrv) parts = g_strsplit (delta_name, "-", 2);
*out_from = *out_to = NULL;
if (parts[0] && parts[1])