Fix zero-argument :qmarks form. Closes issue #26.

common/content/commandline.js

@@ -18,8 +18,8 @@ const CommandWidgets = Class("CommandWidgets", {
         });
         this.addElem({
             name: "commandline",
-            getValue: function () this.command,
             getGroup: function () options.get("guioptions").has("C") ? this.commandbar : this.statusbar,
+            getValue: function () this.command,
             noValue: true,
         });
         this.addElem({
@@ -129,7 +129,7 @@ const CommandWidgets = Class("CommandWidgets", {
                 for (let group in values([this.commandbar, this.statusbar])) {
                     let meth, node = group[elem.name];
                     let visible = (value && group === activeGroup);
-                    if (!node.collapsed == !visible) {
+                    if (node && !node.collapsed == !visible) {
                         node.collapsed = !visible;
                         if (elem.onVisibility)
                             elem.onVisibility.call(this, node, visible);

common/content/completion.js

@@ -442,6 +442,7 @@ const CompletionContext = Class("CompletionContext", {
             this.processor[0] = function (item, text) self.process[0].call(self, item,
                     template.highlightFilter(item.text, self.filter));
 
+        try {
             // Item prototypes
             let proto = this.proto;
             if (!this.cache.constructed)
@@ -455,7 +456,14 @@ const CompletionContext = Class("CompletionContext", {
             // Sorting
             if (this.sortResults && this.compare)
                 filtered.sort(this.compare);
+
             return this.cache.filtered = filtered;
+        }
+        catch (e) {
+            this.message = "Error: " + e;
+            dactyl.reportError(e);
+            return [];
+        }
     },
 
     get substrings() {

common/content/quickmarks.js

@@ -157,7 +157,8 @@ const QuickMarks = Module("quickmarks", {
         commands.add(["qmarks"],
             "Show all QuickMarks",
             function (args) {
-                args = args[0];
+                dactyl.dump(args);
+                args = args[0] || "";
 
                 // ignore invalid qmark characters unless there are no valid qmark chars
                 dactyl.assert(!args || /[a-zA-Z0-9]/.test(args), "E283: No QuickMarks matching " + args.quote());

pentadactyl/NEWS

@@ -41,6 +41,7 @@
     * Added 'autocomplete' option for specifying which completion
       groups should be auto-completed.
     * Added 'banghist' option.
+    * Added BookmarkChange, BookmarkRemove autocommands.
     * Added -keyword, -tags, -title to :delbmarks
     * Added 'hintkeys' option.
     * Replaced 'focuscontent' with 'strictfocus'.

pentadactyl/TODO

@@ -25,11 +25,6 @@ BUGS:
 (recent Mercurial regressions):
 9 about:pentadactyl is currently about:undefined
 
-- messages is still broken in several ways - needs testing.
-  => it often overwrites the open command line while editing etc.
-- :messages is _very_ slow for message history of several thousand lines ->
-  Unresponsive Script: util.js:79 (sometimes xmlToDom() and elsewhere)
-
 FEATURES:
 9 Add quoting help tag
 8 Document Caret and Visual modes.