Removes unused hasOwn var declaration. Fixes #9510 #404

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@rwaldron
Member

rwaldron commented Jun 3, 2011

No description provided.

@mikesherov

This comment has been minimized.

Show comment
Hide comment
@mikesherov

mikesherov Jun 4, 2011

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?

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?

This comment has been minimized.

Show comment
Hide comment
@rwaldron

rwaldron Jun 4, 2011

Owner

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

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

This comment has been minimized.

Show comment
Hide comment
@mikesherov

mikesherov Jun 4, 2011

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!

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!

This comment has been minimized.

Show comment
Hide comment
@rwaldron

rwaldron Jun 4, 2011

Owner

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

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

@timmywil

This comment has been minimized.

Show comment
Hide comment
@timmywil

timmywil Jun 7, 2011

Member

Landed in commit c3c001c.

Member

timmywil commented Jun 7, 2011

Landed in commit c3c001c.

@timmywil timmywil closed this Jun 7, 2011

@SaraJo SaraJo referenced this pull request Oct 29, 2011

Closed

Patch for ticket #9830 #567

mescoda pushed a commit to mescoda/jquery that referenced this pull request Nov 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment