Bug Tracker

Opened 10 years ago

Closed 10 years ago

#4399 closed bug (fixed)

Unused variables in dimensions.js should be removed [PATCH]

Reported by: CraigBuchek Owned by: brandon
Priority: trivial Milestone: 1.4
Component: dimensions Version: 1.3.2
Keywords: Cc:
Blocked by: Blocking:

Description

Browsing through the source, I noticed that the tl and br variables are defined but never used.

I've tested the included patch on Safari 3.2.1 and Firefox 3.0.7 on Mac OS X.

Saves 94 bytes!

Attachments (1)

jquery.4399.diff (571 bytes) - added by CraigBuchek 10 years ago.
Remove unused variables in dimensions.js

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by CraigBuchek

Attachment: jquery.4399.diff added

Remove unused variables in dimensions.js

comment:1 Changed 10 years ago by brandon

Resolution: fixed
Status: newclosed

Fixed in r6296

Thanks for the patch. :)

Note: See TracTickets for help on using tickets.