Give feedback if page is added to bookmarks.
authorDaniel Carl <danielcarl@gmx.de>
Mon, 8 Jul 2013 17:32:01 +0000 (19:32 +0200)
committerDaniel Carl <danielcarl@gmx.de>
Mon, 8 Jul 2013 17:32:01 +0000 (19:32 +0200)
src/bookmark.c
src/bookmark.h
src/command.c

index 7a38ad9..9ef4ba1 100644 (file)
@@ -34,7 +34,7 @@ static void free_bookmark(Bookmark *bm);
 /**
  * Write a new bookmark entry to the end of bookmark file.
  */
-void bookmark_add(const char *uri, const char *tags)
+gboolean bookmark_add(const char *uri, const char *tags)
 {
     FILE *f;
 
@@ -49,7 +49,10 @@ void bookmark_add(const char *uri, const char *tags)
 
         file_lock_set(fileno(f), F_UNLCK);
         fclose(f);
+
+        return true;
     }
+    return false;
 }
 
 /**
index 20f1538..0404601 100644 (file)
@@ -20,7 +20,7 @@
 #ifndef _BOOKMARK_H
 #define _BOOKMARK_H
 
-void bookmark_add(const char *uri, const char *tags);
+gboolean bookmark_add(const char *uri, const char *tags);
 GList *bookmark_get_by_tags(const char *tags);
 
 #endif /* end of include guard: _BOOKMARK_H */
index c5630ae..02aedf1 100644 (file)
@@ -703,8 +703,12 @@ gboolean command_bookmark(const Arg *arg)
 {
     vb_set_mode(VB_MODE_NORMAL, false);
 
-    bookmark_add(GET_URI(), arg->s);
-    return true;
+    if (bookmark_add(GET_URI(), arg->s)) {
+        vb_echo_force(VB_MSG_NORMAL, false, "Bookmark added");
+
+        return true;
+    }
+    return false;
 }
 
 gboolean command_eval(const Arg *arg)