Moved some RegExp in core.js, goes toward #4111.

This commit is contained in:
John Resig 2009-07-19 19:47:30 +00:00
parent eaca153aea
commit 03de2cb5e7

View file

@ -25,6 +25,12 @@ var jQuery = function( selector, context ) {
// Is it a simple selector // Is it a simple selector
isSimple = /^.[^:#\[\.,]*$/, isSimple = /^.[^:#\[\.,]*$/,
// Check if a string has a non-whitespace character in it
rnotwhite = /\S/,
// Used for trimming whitespace
rtrim = /^\s+|\s+$/g,
// Keep a UserAgent string for use with jQuery.browser // Keep a UserAgent string for use with jQuery.browser
userAgent = navigator.userAgent.toLowerCase(), userAgent = navigator.userAgent.toLowerCase(),
@ -303,7 +309,7 @@ jQuery.extend({
// Evalulates a script in a global context // Evalulates a script in a global context
globalEval: function( data ) { globalEval: function( data ) {
if ( data && /\S/.test(data) ) { if ( data && rnotwhite.test(data) ) {
// Inspired by code by Andrea Giammarchi // Inspired by code by Andrea Giammarchi
// http://webreflection.blogspot.com/2007/08/global-scope-evaluation-and-dom.html // http://webreflection.blogspot.com/2007/08/global-scope-evaluation-and-dom.html
var head = document.getElementsByTagName("head")[0] || document.documentElement, var head = document.getElementsByTagName("head")[0] || document.documentElement,
@ -366,7 +372,7 @@ jQuery.extend({
}, },
trim: function( text ) { trim: function( text ) {
return (text || "").replace( /^\s+|\s+$/g, "" ); return (text || "").replace( rtrim, "" );
}, },
makeArray: function( array ) { makeArray: function( array ) {