Inverted character classes

- Moved node class declarations into .h file where they belong.
- Implemented adding of inverted character classes to dispatch tables.
- Cleaned sorting up a bit and fixed a windows build problem.

[Erik Corry, 2008-11-13 11:45:42]

LGTM

http://codereview.chromium.org/10682/diff/1/3
File src/jsregexp.cc (right):

http://codereview.chromium.org/10682/diff/1/3#newcode1545
Line 1545: table->AddRange(CharacterRange(last, 0xFFFF), index);
I think this goes wrong if range.to() for the last range is 0xfffe.  I imagine
there will be literally trillions of regexps with ranges ending in one, but not
the other BOM symbol, so it's important to get this right!

http://codereview.chromium.org/10682/diff/1/7
File src/utils.h (right):

http://codereview.chromium.org/10682/diff/1/7#newcode87
Line 87: static int Spaceship(const T& a, const T& b) {
The reasoning behind this name is a little opaque to me.  Also it seems a pity
that it is only ever used for uc16s where 'return a - b' would be faster.

[Christian Plesner Hansen, 2008-11-13 11:58:36]

http://codereview.chromium.org/10682/diff/1/3
File src/jsregexp.cc (right):

http://codereview.chromium.org/10682/diff/1/3#newcode1545
Line 1545: table->AddRange(CharacterRange(last, 0xFFFF), index);
Good point.  I've fixed it and added a test.

http://codereview.chromium.org/10682/diff/1/7
File src/utils.h (right):

http://codereview.chromium.org/10682/diff/1/7#newcode87
Line 87: static int Spaceship(const T& a, const T& b) {
http://en.wikipedia.org/wiki/Spaceship_operator

We can make a custom comparator function if it turns out to be a performance
issue.