substring() in v1.9

[Nao]

Hey there,

I noticed that jQuery 1.9 (and 1.8, maybe earlier, not in 1.5 though) has this line:

name = jQuery.camelCase( name.substring(5) );

Given that with a single argument given, substring() behaves the same as substr(), wouldn't it be better to use substr(), which is already used twice in other areas of jQuery? This would ensure optimal compression...

Very minor thing, I know, but a byte is a byte :)

PS: also noticed the copyright date in jquery-1.9.js is 2012, instead of 2013. I suppose someone will notice at some point anyway!

[dmethvin]

Agreed on both points, thanks!

[gibson042]

1.9: ​https://github.com/jquery/jquery/commit/37eb6125c0fb0bec4e0d043bc78110b4f49f3aa6 2.0: ​https://github.com/jquery/jquery/commit/8871218cdf37fa0a3e0457882828648e6a40164a

[Nao]

Thank you for committing the fix :) I looked at it and saw you replaced with a .slice(), which makes more sense now that I've read the MDN page for that, eh.

While I'm at it then -- why not change the two remaining .substr() calls in jQuery to .slice()?

They will both accept the change without any issues, there's not even the need to update parameters. Actually, if I'm not mistaken, "result.substr( result.length - check.length )" could be shortened to "result.slice( -check.length )", but I don't know if it compresses as well.

[gibson042]

The referenced commits include Sizzle updates that address the remaining substr calls, and in fact the subtraction operation was replaced with a negative index.

[Nao]

Ah... Yes indeed! Sorry, I didn't realize Sizzle was in its own repo ;)

​https://github.com/jquery/sizzle/commit/200c8724121819f4a6b1f1d155ef9361d0a27518

Then all is good, thanks again for your time!