Fix RegExp white-space character class to match BOMs.

src/jsregexp.cc

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 3579 matching lines @@
   printer.PrintNode(label, node);
 }
 
 
 #endif  // DEBUG
 
 
 // -------------------------------------------------------------------
 // Tree to graph conversion
 
-static const int kSpaceRangeCount = 20;
-static const int kSpaceRangeAsciiCount = 4;
+static const int kSpaceRangeCount = 22;

[rossberg, 2012/02/20 16:03:19]

Not your code, but can't we compute this constant using ARRAY_SIZE?

[Michael Starzinger, 2012/02/20 16:34:00]

Done.

 static const uc16 kSpaceRanges[kSpaceRangeCount] = { 0x0009, 0x000D, 0x0020,
     0x0020, 0x00A0, 0x00A0, 0x1680, 0x1680, 0x180E, 0x180E, 0x2000, 0x200A,
-    0x2028, 0x2029, 0x202F, 0x202F, 0x205F, 0x205F, 0x3000, 0x3000 };
+    0x2028, 0x2029, 0x202F, 0x202F, 0x205F, 0x205F, 0x3000, 0x3000, 0xFEFF,
+    0xFEFF };
 
 static const int kWordRangeCount = 8;
 static const uc16 kWordRanges[kWordRangeCount] = { '0', '9', 'A', 'Z', '_',
     '_', 'a', 'z' };
 
 static const int kDigitRangeCount = 2;
 static const uc16 kDigitRanges[kDigitRangeCount] = { '0', '9' };
 
 static const int kLineTerminatorRangeCount = 6;
 static const uc16 kLineTerminatorRanges[kLineTerminatorRangeCount] = { 0x000A,
@@ skipping 1719 matching lines @@
   }
 
   return compiler.Assemble(&macro_assembler,
                            node,
                            data->capture_count,
                            pattern);
 }
 
 
 }}  // namespace v8::internal