From 996300c782267e94692935e246ecc5611141b1d5 Mon Sep 17 00:00:00 2001
From: Doug Kearns <dougkearns@gmail.com>
Date: Thu, 21 Jul 2011 02:29:28 +1000
Subject: [PATCH] Fix :delgroup completer's builtin group exclusions.

---
 common/modules/contexts.jsm             | 2 +-
 common/tests/functional/testCommands.js | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/modules/contexts.jsm b/common/modules/contexts.jsm
index 95939716..f5d976ca 100644
--- a/common/modules/contexts.jsm
+++ b/common/modules/contexts.jsm
@@ -585,8 +585,8 @@ var Contexts = Module("contexts", {
                 completer: function (context, args) {
                     if (args.bang)
                         return;
-                    modules.completion.group(context);
                     context.filters.push(function ({ item }) !item.builtin);
+                    modules.completion.group(context);
                 }
             });
 
diff --git a/common/tests/functional/testCommands.js b/common/tests/functional/testCommands.js
index 86750083..c65365ac 100644
--- a/common/tests/functional/testCommands.js
+++ b/common/tests/functional/testCommands.js
@@ -184,7 +184,7 @@ var tests = {
         }
     ],
     delgroup: {
-        error: ["!", "", "builtin"],
+        error: ["", "! foo", "builtin"],
         completions: [""]
     },
     delmacros: {