Permalink
Browse files

Removes unused hasOwn var declaration. Fixes #9510

  • Loading branch information...
1 parent 1d202c6 commit 821deda587b1398a5afb09481d489c2a7f42093e @rwaldron committed Jun 3, 2011
Showing with 1 addition and 2 deletions.
  1. +1 −2 src/event.js
View
@@ -1,7 +1,6 @@
(function( jQuery ) {
-var hasOwn = Object.prototype.hasOwnProperty,
- rnamespaces = /\.(.*)$/,
+var rnamespaces = /\.(.*)$/,
rformElems = /^(?:textarea|input|select)$/i,
rperiod = /\./g,
rspaces = / /g,

4 comments on commit 821deda

This file, events.js has a whole bunch of other stylistic issues, including but not limited to, multiple vat declarations per function, var declarations in loops, unused variables, and whitespacing. I assume it hasn't really been touched in a bit?

Owner

rwaldron replied Jun 4, 2011

yeah bro.. It's on the rewrite list for 1.7 - Dave Methvin has some great plans for cleaning this up

Cool. Just curious, is that on the bug tracker, or listed somewhere? I was going to start doing that cleanup myself, and I would rather work on something that one of the core members isn't. Thanks again!

Owner

rwaldron replied Jun 4, 2011

Dave Methvin is doing the major re-write tasks http://bugs.jquery.com/ticket/9393 - the man has a plan! :D

Please sign in to comment.