src: remove --noharmony_classes again
authorBen Noordhuis <info@bnoordhuis.nl>
Sat, 17 Jan 2015 22:52:52 +0000 (23:52 +0100)
committerBen Noordhuis <info@bnoordhuis.nl>
Sun, 18 Jan 2015 12:09:23 +0000 (13:09 +0100)
Now that V8 has been upgraded, remove the --noharmony_classes and
--noharmony_object_literals workarounds from commits a2751e3e and
4e58211b.

PR-URL: https://github.com/iojs/io.js/pull/490
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed-By: Kenan Sulayman <kenan@sly.mn>
Reviewed-By: Rod Vagg <rod@vagg.org>
src/node.cc

index 110fd9f..caf1215 100644 (file)
@@ -3337,14 +3337,6 @@ void Init(int* argc,
                 DispatchDebugMessagesAsyncCallback);
   uv_unref(reinterpret_cast<uv_handle_t*>(&dispatch_debug_messages_async));
 
-  // TODO(bnoordhuis) V8 3.32 is unshipping Harmony classes for the moment.
-  // We're currently at 3.31, disable classes for feature parity.  Remove
-  // again when we upgrade.
-  V8::SetFlagsFromString("--noharmony_classes",
-                         sizeof("--noharmony_classes") - 1);
-  V8::SetFlagsFromString("--noharmony_object_literals",
-                         sizeof("--noharmony_object_literals") - 1);
-
 #if defined(NODE_V8_OPTIONS)
   // Should come before the call to V8::SetFlagsFromCommandLine()
   // so the user can disable a flag --foo at run-time by passing