Opened 12 years ago
Closed 11 years ago
#3194 closed enhancement (invalid)
Possible optimization for the selector module
Reported by: | flesler | Owned by: | flesler |
---|---|---|---|
Priority: | minor | Milestone: | 1.3 |
Component: | selector | Version: | 1.2.6 |
Keywords: | attr | Cc: | |
Blocked by: | Blocking: |
Description
This reduces the code size and makes things more straight forward. But could be slower than the actual code...
Attachments (1)
Change History (6)
Changed 12 years ago by
Attachment: | selector[5795].diff added |
---|
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
The performance overhead isn't worth the code improvement.
comment:3 Changed 12 years ago by
Resolution: | invalid |
---|---|
Status: | closed → reopened |
The code to be removed is redundant with attr, maybe a refactoring is a better approach.
comment:4 Changed 12 years ago by
I don't see why "redundant". Using attr() is the logical approach, it just adds too much overhead.
comment:5 Changed 11 years ago by
Resolution: | → invalid |
---|---|
Status: | reopened → closed |
This doesn't apply anymore since Sizzle was introduced.
Note: See
TracTickets for help on using
tickets.
This adds a LOT of overhead :(