Bug Tracker

Ticket #7629 (closed enhancement: worksforme)

Opened 4 years ago

Last modified 4 years ago

Extend coverage of stored references to core methods metaticket

Reported by: rwaldron Owned by:
Priority: low Milestone:
Component: misc Version: 1.4.4
Keywords: Cc:
Blocking: Blocked by:

Description (last modified by rwaldron) (diff)

A comment on #6809 pointed to a patch that does some optimizations by extended the idea already in practice of storing references to core methods, strings and other "constant" type information that is frequently reused.

A VERY good resource for this practice was published by the winner of the JS1k contest, here:

 http://marijnhaverbeke.nl/js1k/

Additionally, this a link to the patch:

 https://github.com/monsterzz/jquery/commit/7ab7f285127fe29107a47fe486e3ea40ff51aec9

Change History

comment:1 Changed 4 years ago by rwaldron

  • Priority changed from undecided to low
  • Component changed from unfiled to misc
  • Description modified (diff)

comment:2 Changed 4 years ago by rwaldron

  • Status changed from new to open

comment:3 Changed 4 years ago by john

  • Status changed from open to closed
  • Resolution set to worksforme

"storing references to core methods, strings and other "constant" type information that is frequently reused."

Don't we do this already? We store reference to many DOM methods, we cache all RegExp and functions at the top of our code. I think we're already OK here.

I've moved the commit above into #6809.

Note: See TracTickets for help on using tickets.