From ffd2da55bd38823b0962450c1ea79afd39d932e5 Mon Sep 17 00:00:00 2001
From: Moritz Lipp <mlq@pwmt.org>
Date: Tue, 21 Feb 2012 17:11:59 +0100
Subject: [PATCH] Style changes and additional parseable arguments

---
 config.c    | 42 +++++++++++++++++++++++-------------------
 shortcuts.c |  8 ++++----
 2 files changed, 27 insertions(+), 23 deletions(-)

diff --git a/config.c b/config.c
index 402cdb2..343e22d 100644
--- a/config.c
+++ b/config.c
@@ -231,7 +231,7 @@ config_load_default(zathura_t* zathura)
   girara_shortcut_mapping_add(gsession, "change_mode",       sc_change_mode);
   girara_shortcut_mapping_add(gsession, "follow",            sc_follow);
   girara_shortcut_mapping_add(gsession, "goto",              sc_goto);
-  girara_shortcut_mapping_add(gsession, "index_navigate",    sc_navigate_index);
+  girara_shortcut_mapping_add(gsession, "navigate_index",    sc_navigate_index);
   girara_shortcut_mapping_add(gsession, "navigate",          sc_navigate);
   girara_shortcut_mapping_add(gsession, "quit",              sc_quit);
   girara_shortcut_mapping_add(gsession, "recolor",           sc_recolor);
@@ -246,24 +246,28 @@ config_load_default(zathura_t* zathura)
   girara_shortcut_mapping_add(gsession, "zoom",              sc_zoom);
 
   /* add argument mappings */
-  girara_argument_mapping_add(gsession, "bottom",     BOTTOM);
-  girara_argument_mapping_add(gsession, "default",    DEFAULT);
-  girara_argument_mapping_add(gsession, "down",       DOWN);
-  girara_argument_mapping_add(gsession, "full-down",  FULL_DOWN);
-  girara_argument_mapping_add(gsession, "full-up",    FULL_UP);
-  girara_argument_mapping_add(gsession, "half-down",  HALF_DOWN);
-  girara_argument_mapping_add(gsession, "half-up",    HALF_UP);
-  girara_argument_mapping_add(gsession, "in",         ZOOM_IN);
-  girara_argument_mapping_add(gsession, "left",       LEFT);
-  girara_argument_mapping_add(gsession, "next",       NEXT);
-  girara_argument_mapping_add(gsession, "out",        ZOOM_OUT);
-  girara_argument_mapping_add(gsession, "previous",   PREVIOUS);
-  girara_argument_mapping_add(gsession, "right",      RIGHT);
-  girara_argument_mapping_add(gsession, "specific",   ZOOM_SPECIFIC);
-  girara_argument_mapping_add(gsession, "top",        TOP);
-  girara_argument_mapping_add(gsession, "up",         UP);
-  girara_argument_mapping_add(gsession, "best-fit",   ADJUST_BESTFIT);
-  girara_argument_mapping_add(gsession, "width",      ADJUST_WIDTH);
+  girara_argument_mapping_add(gsession, "bottom",       BOTTOM);
+  girara_argument_mapping_add(gsession, "default",      DEFAULT);
+  girara_argument_mapping_add(gsession, "collapse",     COLLAPSE);
+  girara_argument_mapping_add(gsession, "collapse-all", COLLAPSE_ALL);
+  girara_argument_mapping_add(gsession, "down",         DOWN);
+  girara_argument_mapping_add(gsession, "expand",       EXPAND);
+  girara_argument_mapping_add(gsession, "expand-all",   EXPAND_ALL);
+  girara_argument_mapping_add(gsession, "full-down",    FULL_DOWN);
+  girara_argument_mapping_add(gsession, "full-up",      FULL_UP);
+  girara_argument_mapping_add(gsession, "half-down",    HALF_DOWN);
+  girara_argument_mapping_add(gsession, "half-up",      HALF_UP);
+  girara_argument_mapping_add(gsession, "in",           ZOOM_IN);
+  girara_argument_mapping_add(gsession, "left",         LEFT);
+  girara_argument_mapping_add(gsession, "next",         NEXT);
+  girara_argument_mapping_add(gsession, "out",          ZOOM_OUT);
+  girara_argument_mapping_add(gsession, "previous",     PREVIOUS);
+  girara_argument_mapping_add(gsession, "right",        RIGHT);
+  girara_argument_mapping_add(gsession, "specific",     ZOOM_SPECIFIC);
+  girara_argument_mapping_add(gsession, "top",          TOP);
+  girara_argument_mapping_add(gsession, "up",           UP);
+  girara_argument_mapping_add(gsession, "best-fit",     ADJUST_BESTFIT);
+  girara_argument_mapping_add(gsession, "width",        ADJUST_WIDTH);
 }
 
 void
diff --git a/shortcuts.c b/shortcuts.c
index 7421d1a..e1cd346 100644
--- a/shortcuts.c
+++ b/shortcuts.c
@@ -584,7 +584,7 @@ sc_navigate_index(girara_session_t* session, girara_argument_t* argument,
 
   switch(argument->n) {
     case UP:
-      if(gtk_tree_path_prev(path) == FALSE) {
+      if (gtk_tree_path_prev(path) == FALSE) {
         is_valid_path = gtk_tree_path_up(path);
       } else { /* row above */
         while(gtk_tree_view_row_expanded(tree_view, path)) {
@@ -598,14 +598,14 @@ sc_navigate_index(girara_session_t* session, girara_argument_t* argument,
       }
       break;
     case COLLAPSE:
-      if(!gtk_tree_view_collapse_row(tree_view, path)
+      if (gtk_tree_view_collapse_row(tree_view, path) == FALSE
         && gtk_tree_path_get_depth(path) > 1) {
         gtk_tree_path_up(path);
         gtk_tree_view_collapse_row(tree_view, path);
       }
       break;
     case DOWN:
-      if(gtk_tree_view_row_expanded(tree_view, path)) {
+      if (gtk_tree_view_row_expanded(tree_view, path) == TRUE) {
         gtk_tree_path_down(path);
       } else {
         do {
@@ -619,7 +619,7 @@ sc_navigate_index(girara_session_t* session, girara_argument_t* argument,
       }
       break;
     case EXPAND:
-      if(gtk_tree_view_expand_row(tree_view, path, FALSE)) {
+      if (gtk_tree_view_expand_row(tree_view, path, FALSE)) {
         gtk_tree_path_down(path);
       }
       break;