diff options
author | Alberto Mardegan <mardy@users.sourceforge.net> | 2013-04-02 18:46:41 +0300 |
---|---|---|
committer | Henrik Brautaset Aronsen <subsurface@henrik.synth.no> | 2013-04-07 17:37:09 +0200 |
commit | 68119c5b5415c2797e11356dc8db63bd84c577cf (patch) | |
tree | 69984129fe914f3cae60523227fc20f1c62eb173 /main.c | |
parent | 0b8462bd584c7606e3a9091947ccdbf04a32dea8 (diff) | |
download | subsurface-68119c5b5415c2797e11356dc8db63bd84c577cf.tar.gz |
Move set_filename() calls outside of parse_file()
Remove the boolean parameter from parse_file; the code is more readable
by having an explicit call to set_filename() where necessary, rather
than a boolean parameter.
Signed-off-by: Henrik Brautaset Aronsen <subsurface@henrik.synth.no>
Signed-off-by: Alberto Mardegan <mardy@users.sourceforge.net>
Diffstat (limited to 'main.c')
-rw-r--r-- | main.c | 10 |
1 files changed, 7 insertions, 3 deletions
@@ -348,8 +348,12 @@ int main(int argc, char **argv) /* if we have exactly one filename, parse_file will set * that to be the default. Otherwise there will be no default filename */ set_filename(NULL, TRUE); - parse_file(a, &error, no_filenames); - no_filenames = FALSE; + parse_file(a, &error); + if (no_filenames) + { + set_filename(a, TRUE); + no_filenames = FALSE; + } if (error != NULL) { report_error(error); @@ -360,7 +364,7 @@ int main(int argc, char **argv) if (no_filenames) { GError *error = NULL; const char *filename = prefs.default_filename; - parse_file(filename, &error, TRUE); + parse_file(filename, &error); /* don't report errors - this file may not exist, but make sure we remember this as the filename in use */ set_filename(filename, FALSE); |