diff options
author | Daniel Friesel <derf@finalrewind.org> | 2011-08-29 22:39:49 +0200 |
---|---|---|
committer | Daniel Friesel <derf@finalrewind.org> | 2011-08-29 22:39:57 +0200 |
commit | 6206ca48113d6790e5681d98ef2b89fd2a1d44da (patch) | |
tree | e273629e7f5e16704e9713076771f54eeda73051 | |
parent | 6476d7dc4e3acc63aeb57b6d89f50c9b44a2ec98 (diff) |
Various code fixups
* remove unused variable Im1 in filelist.c
* options are initialised as 0, so only explicitly set those to 0 for which it
makes sense because of their context
* Make a few warnings about wrong option combinations fatal
-rw-r--r-- | src/filelist.c | 1 | ||||
-rw-r--r-- | src/keyevents.c | 2 | ||||
-rw-r--r-- | src/options.c | 16 |
3 files changed, 4 insertions, 15 deletions
diff --git a/src/filelist.c b/src/filelist.c index d83b8df..215f371 100644 --- a/src/filelist.c +++ b/src/filelist.c @@ -462,7 +462,6 @@ gib_list *feh_read_filelist(char *filename) FILE *fp; gib_list *list = NULL; char s[1024], s1[1024]; - Imlib_Image im1; if (!filename) return(NULL); diff --git a/src/keyevents.c b/src/keyevents.c index 82d0cce..60663db 100644 --- a/src/keyevents.c +++ b/src/keyevents.c @@ -65,7 +65,7 @@ static void feh_set_parse_kb_partial(fehkey *key, int index, char *ks) { mod |= Mod4Mask; break; default: - weprintf("keys: invalid modifier %c in %s", cur[0], ks); + weprintf("keys: invalid modifier %c in \"%s\"", cur[0], ks); break; } cur += 2; diff --git a/src/options.c b/src/options.c index c5ed85d..fe0cf87 100644 --- a/src/options.c +++ b/src/options.c @@ -70,8 +70,6 @@ void init_parse_options(int argc, char **argv) opt.prev_button = 4; opt.next_button = 5; - opt.draw_actions = 0; - opt.rotate_button = 2; opt.no_rotate_ctrl_mask = 0; opt.blur_button = 1; @@ -80,7 +78,6 @@ void init_parse_options(int argc, char **argv) opt.start_list_at = NULL; opt.jump_on_resort = 1; - opt.xinerama = 0; opt.screen_clip = 1; #ifdef HAVE_LIBXINERAMA /* if we're using xinerama, then enable it by default */ @@ -823,15 +820,11 @@ static void check_options(void) } if (opt.full_screen && opt.multiwindow) { - weprintf("you shouldn't combine multiwindow mode with full-screen mode,\n" - " Multiwindow mode has been disabled."); - opt.multiwindow = 0; + eprintf("You cannot combine --full-screen with --multiwindow"); } if (opt.list && (opt.multiwindow || opt.index || opt.collage)) { - weprintf("list mode can't be combined with other processing modes,\n" - " list mode disabled."); - opt.list = 0; + eprintf("You cannot combine --list with other modes"); } if (opt.sort && opt.randomize) { @@ -841,10 +834,7 @@ static void check_options(void) } if (opt.loadables && opt.unloadables) { - weprintf("You cant show loadables AND unloadables...\n" - "you might as well use ls ;)\n" - "loadables only will be shown\n"); - opt.unloadables = 0; + eprintf("You cannot combine --loadable with --unloadable"); } if (opt.thumb_title && (!opt.thumbs)) { |