mirror of
https://github.com/profanity-im/profanity.git
synced 2024-12-04 14:46:46 -05:00
Use null check convention in theme.c
This commit is contained in:
parent
b093b99c15
commit
4ba33005d0
@ -160,7 +160,7 @@ _theme_load_file(const char * const theme_name)
|
|||||||
{
|
{
|
||||||
// use default theme
|
// use default theme
|
||||||
if (theme_name == NULL || strcmp(theme_name, "default") == 0) {
|
if (theme_name == NULL || strcmp(theme_name, "default") == 0) {
|
||||||
if (theme != NULL) {
|
if (theme) {
|
||||||
g_key_file_free(theme);
|
g_key_file_free(theme);
|
||||||
}
|
}
|
||||||
theme = g_key_file_new();
|
theme = g_key_file_new();
|
||||||
@ -173,12 +173,12 @@ _theme_load_file(const char * const theme_name)
|
|||||||
return FALSE;
|
return FALSE;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (theme_loc != NULL) {
|
if (theme_loc) {
|
||||||
g_string_free(theme_loc, TRUE);
|
g_string_free(theme_loc, TRUE);
|
||||||
}
|
}
|
||||||
theme_loc = new_theme_file;
|
theme_loc = new_theme_file;
|
||||||
log_info("Loading theme \"%s\"", theme_name);
|
log_info("Loading theme \"%s\"", theme_name);
|
||||||
if (theme != NULL) {
|
if (theme) {
|
||||||
g_key_file_free(theme);
|
g_key_file_free(theme);
|
||||||
}
|
}
|
||||||
theme = g_key_file_new();
|
theme = g_key_file_new();
|
||||||
@ -205,10 +205,10 @@ theme_list(void)
|
|||||||
void
|
void
|
||||||
theme_close(void)
|
theme_close(void)
|
||||||
{
|
{
|
||||||
if (theme != NULL) {
|
if (theme) {
|
||||||
g_key_file_free(theme);
|
g_key_file_free(theme);
|
||||||
}
|
}
|
||||||
if (theme_loc != NULL) {
|
if (theme_loc) {
|
||||||
g_string_free(theme_loc, TRUE);
|
g_string_free(theme_loc, TRUE);
|
||||||
}
|
}
|
||||||
if (bold_items) {
|
if (bold_items) {
|
||||||
@ -476,9 +476,9 @@ void
|
|||||||
_theme_list_dir(const gchar * const dir, GSList **result)
|
_theme_list_dir(const gchar * const dir, GSList **result)
|
||||||
{
|
{
|
||||||
GDir *themes = g_dir_open(dir, 0, NULL);
|
GDir *themes = g_dir_open(dir, 0, NULL);
|
||||||
if (themes != NULL) {
|
if (themes) {
|
||||||
const gchar *theme = g_dir_read_name(themes);
|
const gchar *theme = g_dir_read_name(themes);
|
||||||
while (theme != NULL) {
|
while (theme) {
|
||||||
*result = g_slist_append(*result, strdup(theme));
|
*result = g_slist_append(*result, strdup(theme));
|
||||||
theme = g_dir_read_name(themes);
|
theme = g_dir_read_name(themes);
|
||||||
}
|
}
|
||||||
@ -492,7 +492,7 @@ _theme_find(const char * const theme_name)
|
|||||||
GString *path = NULL;
|
GString *path = NULL;
|
||||||
gchar *themes_dir = _get_themes_dir();
|
gchar *themes_dir = _get_themes_dir();
|
||||||
|
|
||||||
if (themes_dir != NULL) {
|
if (themes_dir) {
|
||||||
path = g_string_new(themes_dir);
|
path = g_string_new(themes_dir);
|
||||||
g_free(themes_dir);
|
g_free(themes_dir);
|
||||||
g_string_append(path, "/");
|
g_string_append(path, "/");
|
||||||
|
Loading…
Reference in New Issue
Block a user