refactor ender binding

ext/zest.ender.js

@@ -10,23 +10,20 @@
  * https://github.com/ender-js
  */
 
-// I try not to change the expected functionality here, however,
-// these functions seem like they should belong in a dom library,
-// not as part of the selector engine binding.
-
 ;(function() {
-  var window = this
-    , document = this.document
-    , zest = this.zest.noConflict();
+  var zest = this.zest.noConflict()
+    , document = this.document;
 
   $._select = function(str, context) {
     context = context || document;
+
     if (/^\s*</.test(str)) {
-      context = context.nodeType === 9
-              ? context : context.ownerDocument;
+      context = context.nodeType !== 9
+        ? context.ownerDocument
+        : context;
 
-      var out = []
-        , el = context.createElement('div');
+      var el = context.createElement('div')
+        , out = [];
 
       el.innerHTML = str;
       el = el.firstChild;
@@ -37,32 +34,33 @@
 
       return out;
     }
+
     return zest(str, context);
   };
 
-  $.ender(function() {
-    var has = function(obj, el) {
-      var i = obj.length;
-      while (i--) {
-        if (obj[i] === el) return true;
-      }
-    };
-    return {
-      find: function(sel) {
-        var res = []
-          , i = this.length
-          , r
-          , k;
+  var has = function(obj, el) {
+    var i = obj.length;
+    while (i--) {
+      if (obj[i] === el) return true;
+    }
+  };
+
+  $.ender({
+    find: function(sel) {
+      var res = []
+        , i = this.length
+        , r
+        , k;
 
-        while (i--) {
-          r = zest(sel, this[i]);
-          k = r.length;
-          while (k--) {
-            if (!has(res, r[k])) res.push(r[k]);
-          }
+      while (i--) {
+        r = zest(sel, this[i]);
+        k = r.length;
+        while (k--) {
+          if (!has(res, r[k])) res.push(r[k]);
         }
-        return $(res);
       }
-    };
-  }());
+
+      return $(res);
+    }
+  });
 }).call(this);