Yeah totally agree !!!

There's no point in putting a character group in square brackets too.

You don't need extra space in character class, [\s]+ will do the same

0. It's faster to compare (IMHO) than '==';
1. It's more consistent and obvious;
2. It's recommended by JSLint, for example.

The if statement is redundant. Instead you can just return str === ending because it evaluates to boolean.