diff options
author | Grigori Goronzy <greg@blackbox> | 2010-08-09 03:28:29 +0200 |
---|---|---|
committer | Grigori Goronzy <greg@blackbox> | 2010-08-09 05:58:00 +0200 |
commit | 32083be82b7b533fa85c36ae5ccacea94857fafa (patch) | |
tree | b492d25d52fe7daed512d828aabccaa1eff4352d /libass/ass_fontconfig.c | |
parent | 65d9960f24ec25cadfacd505046f9e4f2932bb12 (diff) | |
download | libass-32083be82b7b533fa85c36ae5ccacea94857fafa.tar.bz2 libass-32083be82b7b533fa85c36ae5ccacea94857fafa.tar.xz |
Get rid of NULL checks for free
The useless "if (foo) free(foo)" idiom is all over the place, just
get rid of it finally...
Diffstat (limited to 'libass/ass_fontconfig.c')
-rw-r--r-- | libass/ass_fontconfig.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/libass/ass_fontconfig.c b/libass/ass_fontconfig.c index a582690..031f4fa 100644 --- a/libass/ass_fontconfig.c +++ b/libass/ass_fontconfig.c @@ -521,14 +521,13 @@ int fontconfig_update(FCInstance *priv) void fontconfig_done(FCInstance *priv) { + + if (priv) { #ifdef CONFIG_FONTCONFIG - if (priv && priv->config) FcConfigDestroy(priv->config); #endif - if (priv && priv->path_default) free(priv->path_default); - if (priv && priv->family_default) free(priv->family_default); - if (priv) - free(priv); + } + free(priv); } |