Bug Tracker

Ticket #3194 (reopened enhancement)

Opened 1 month ago

Last modified 1 month ago

Possible optimization for the selector module

Reported by: flesler Assigned to: flesler
Type: enhancement Priority: minor
Milestone: 1.3 Component: selector
Version: 1.2.6 Keywords: attr
Cc: Needs: Review

Description

This reduces the code size and makes things more straight forward. But could be slower than the actual code...

Attachments

selector[5795].diff (0.6 kB) - added by flesler 1 month ago.

Change History

Changed 1 month ago by flesler

Changed 1 month ago by flesler

This adds a LOT of overhead :(

Changed 1 month ago by joern

  • status changed from new to closed
  • resolution set to invalid

The performance overhead isn't worth the code improvement.

Changed 1 month ago by joern

  • status changed from closed to reopened
  • resolution deleted

The code to be removed is redundant with attr, maybe a refactoring is a better approach.

Changed 1 month ago by flesler

I don't see why "redundant". Using attr() is the logical approach, it just adds too much overhead.

Note: See TracTickets for help on using tickets.