Revision: 3260 Author: [email protected] Date: Tue Nov 10 06:41:53 2009 Log: Modify test using string slices
With slices string string type removed generating a large amount of sub-strings takes more time. Change a test to avoid timeout in debug mode. Review URL: http://codereview.chromium.org/385006 http://code.google.com/p/v8/source/detail?r=3260 Modified: /branches/bleeding_edge/test/mjsunit/deep-recursion.js ======================================= --- /branches/bleeding_edge/test/mjsunit/deep-recursion.js Tue Sep 9 13:08:45 2008 +++ /branches/bleeding_edge/test/mjsunit/deep-recursion.js Tue Nov 10 06:41:53 2009 @@ -30,9 +30,7 @@ * cause stack overflows. */ -var depth = 110000; - -function newdeep(start) { +function newdeep(start, depth) { var d = start; for (var i = 0; i < depth; i++) { d = d + "f"; @@ -40,23 +38,27 @@ return d; } -var deep = newdeep("foo"); +var default_depth = 110000; + +var deep = newdeep("foo", default_depth); assertEquals('f', deep[0]); -var cmp1 = newdeep("a"); -var cmp2 = newdeep("b"); +var cmp1 = newdeep("a", default_depth); +var cmp2 = newdeep("b", default_depth); assertEquals(-1, cmp1.localeCompare(cmp2), "ab"); -var cmp2empty = newdeep("c"); +var cmp2empty = newdeep("c", default_depth); assertTrue(cmp2empty.localeCompare("") > 0, "c"); -var cmp3empty = newdeep("d"); +var cmp3empty = newdeep("d", default_depth); assertTrue("".localeCompare(cmp3empty) < 0), "d"; -var slicer = newdeep("slice"); - -for (i = 0; i < depth + 4; i += 2) { +var slicer_depth = 1100; + +var slicer = newdeep("slice", slicer_depth); + +for (i = 0; i < slicer_depth + 4; i += 2) { slicer = slicer.slice(1, -1); } --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
