curCSS Really slow

[malfist]

I was running firebug's profiler over my webapp and found that the function using the most processor time (although it does not have the longest execution time) is curCSS. I dug around in the source code of Jquery and found it. It's a helper method and it plays with the DOM, the slowest part of javascript. Considering most good programmers don't need this (they do it already), it's redundant and a pointless waste.

Could this be changed to an opt-in function instead of a default?

[malfist]

The profile log.

[flesler]

curCSS is used a LOT all over the source. It's not just the external use, it's widely used internally for dimensions, animations, offset, etc. And it cannot be optional... it's totally necessary.

[malfist]

I commented out it's entire contents and had no adverse effects. I gained considerable increase in responce time.

[flesler]

Well.. I was curious about this, commented all the function and ran the test suite.

100 tests of 1134 failed.

They belong to fx, dimensions, css, width/height and a few more single tests.

This shows that the function isn't used everywhere, but only by certain modules. You probably don't use them so it doesn't fail (or maybe it does fail and you don't notice).

Anyway.. all the mentioned modules are included into the core, and need to be supported.

You can always make your own build of jQuery, removing w/e you prefer.

Thanks