diff options
author | Jon Lund Steffensen <jonlst@gmail.com> | 2015-05-10 19:43:05 -0400 |
---|---|---|
committer | Jon Lund Steffensen <jonlst@gmail.com> | 2015-05-10 19:43:05 -0400 |
commit | 6c51f4f7b07fdefd5c69f6d7c96dc2f618f15189 (patch) | |
tree | 524c0a993e4273bdab59cb786526b62581dfb4db /src/redshift.c | |
parent | Fix #174: Use nanosleep() instead of usleep() (diff) | |
parent | remove unnecessary null-checks, it is safe to pass NULL to free (per documentation) (diff) | |
download | redshift-ng-6c51f4f7b07fdefd5c69f6d7c96dc2f618f15189.tar.gz redshift-ng-6c51f4f7b07fdefd5c69f6d7c96dc2f618f15189.tar.bz2 redshift-ng-6c51f4f7b07fdefd5c69f6d7c96dc2f618f15189.tar.xz |
Merge pull request #214 from maandree/null-checks
Remove unnecessary null-checks
Diffstat (limited to 'src/redshift.c')
-rw-r--r-- | src/redshift.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/redshift.c b/src/redshift.c index 0fcb0ba..095e0db 100644 --- a/src/redshift.c +++ b/src/redshift.c @@ -1096,7 +1096,7 @@ main(int argc, char *argv[]) &scheme.night.brightness); break; case 'c': - if (config_filepath != NULL) free(config_filepath); + free(config_filepath); config_filepath = strdup(optarg); break; case 'g': @@ -1245,7 +1245,7 @@ main(int argc, char *argv[]) exit(EXIT_FAILURE); } - if (config_filepath != NULL) free(config_filepath); + free(config_filepath); /* Read global config settings. */ config_ini_section_t *section = config_ini_get_section(&config_state, |