zeroneta
Jan 5th

  http://bgscript.com/jscore/script/core.js

zeroneta
Jan 5th

_.trim = function( a, s )
{
	s = s == un ? '\s\n\r\t\f\x0b\xa0\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200a\u200b\u2028\u2029\u3000' : real( s );
	return ( a + '' ).replace( new _._.RegExp( '(^[' + s + ']*)|([' + s + ']*$)', 'g' ), '' );
}

$.real = function( a )
{
	return ( a + '' ).replace( /(\$|\^|\*|\(|\)|\-|\+|\||\\|\{|\[|\}|\]|\,|\.|\?|\/)/g, '\\$1' );
}

Kevin van Zonneveld
6 Oct '08

  @ Onno Marsmann: Fixed, thank you!

Onno Marsman
4 Oct '08

  Something like trim(16,1) does not work correctly.

Fix:
Add the following lines of code to the beginning of the function:
[CODE="Javascript"]
str += '';
charlist += '';
[/CODE]

Kevin van Zonneveld
27 Aug '08

  @ Jack: That's a good idea, thank you, I've implemented it and credited you accordingly.

@ Ben W: I will have to study on that, thanks for the input

Kevin van Zonneveld
27 Aug '08

  @ 0x0: That's definitely something to pounder about. We did not in the first place, because one of the project goals is to keep functions as standalone as reasonably possible. This duplication seemed reasonable for a standalone trim.

Ben W
27 Aug '08

  Wouldn't it be better to use regex? Like one of these:

http://blog.stevenlevithan.com/archives/faster-trim-javascript

Jack
21 Aug '08

  we can avoid unnecessary string length computations by changing the "for" loop from

[CODE="Javascript"]
for (var i = 0; i < str.length; i++)
[/CODE]

to

[CODE="Javascript"]
for (var i = 0, l=str.length; i < l; i++)
[/CODE]

[CODE="Javascript"]
function trim( str, charlist ) {


var whitespace;

if (!charlist) {
whitespace = ' \n\r\t\f\x0b\xa0\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200a\u200b\u2028\u2029\u3000';
} else {
whitespace = charlist.replace(/([\[\]\(\)\.\?\/\*\{\}\+\$\^\:])/g, '\$1');
}

for (var i = 0, l=str.length; i < l; i++) {
if (whitespace.indexOf(str.charAt(i)) === -1) {
str = str.substring(i);
break;
}
}
for (i = str.length - 1; i >= 0; i--) {
if (whitespace.indexOf(str.charAt(i)) === -1) {
str = str.substring(0, i + 1);
break;
}
}
return whitespace.indexOf(str.charAt(0)) === -1 ? str : '';
}
[/CODE]

0x0
7 Aug '08

  If you also have rtrim and ltrim, you could just do this:
[CODE="Javascript"]
function trim(str, charlist) {
return ltrim(rtrim(str, charlist), charlist);
}
[/CODE]

DxGx
20 Mar '08

  [CODE="Javascript"]
function trim (str) {
var whitespace = ' \n\r\t\f\x0b\xa0\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200a\u200b\u2028\u2029\u3000';
for (var i = 0; i < str.length; i++) {
if (whitespace.indexOf(str.charAt(i)) === -1) {
str = str.substring(i);
break;
}
}
for (i = str.length - 1; i >= 0; i--) {
if (whitespace.indexOf(str.charAt(i)) === -1) {
str = str.substring(0, i + 1);
break;
}
}
return whitespace.indexOf(str.charAt(0)) === -1 ? str : '';
}
[/CODE]

Replace trim function by this code. Acoding to this site: it is faster: http://blog.stevenlevithan.com/archives/faster-trim-javascript

Kevin van Zonneveld
21 Feb '08

  @ Erkekjetter: Hi Erkekjetter, thanks for your input. I've found the time to recode the functions and updated php.js accordingly.

Erkekjetter
19 Feb '08

  The trim() function for PHP is described as
string trim ( string $str [, string $charlist ] )

In your trim(), rtrim() and ltrim() functions the second parameter charlist is missing. You might change that to have a equivalent function.