Reviewers: Dmitry Lomov (chromium),
Message:
Committed patchset #1 manually as 22834 (tree was closed).
Description:
Fix spurious debug-script assertion after enabling Symbols
This approach, of allowing either N or N+1 in this assert,
is the same approach used before my Symbols patch that naively
changed to use assertEquals.
[email protected]
Committed: https://code.google.com/p/v8/source/detail?r=22834
Please review this at https://codereview.chromium.org/439833002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -1 lines):
M test/mjsunit/debug-script.js
Index: test/mjsunit/debug-script.js
diff --git a/test/mjsunit/debug-script.js b/test/mjsunit/debug-script.js
index
449d2c9b8f75359890568c31bb212691d25cdcff..35bc560c232fba8a8fd26670edec3a9cc56eb093
100644
--- a/test/mjsunit/debug-script.js
+++ b/test/mjsunit/debug-script.js
@@ -59,7 +59,7 @@ for (i = 0; i < scripts.length; i++) {
}
// This has to be updated if the number of native scripts change.
-assertEquals(21, named_native_count);
+assertTrue(named_native_count == 20 || named_native_count == 21);
// Only the 'gc' extension is loaded.
assertEquals(1, extension_count);
// This script and mjsunit.js has been loaded. If using d8, d8 loads
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
