Lgtm! On Tue, Oct 14, 2008 at 12:47 PM, <[EMAIL PROTECTED]> wrote: > Reviewers: Christian Plesner Hansen, > > Message: > Small code review. > Reduces time on my computer from more than 5 minutes to ~1m20s. > > Description: > Reduced the string length and rounds in the extensive indexOf-test in > string-indexof.js. > > Please review this at http://codereview.chromium.org/7138 > > Affected files: > M test/mjsunit/string-indexof.js > > > Index: test/mjsunit/string-indexof.js > diff --git a/test/mjsunit/string-indexof.js b/test/mjsunit/string-indexof.js > index > f732c9e6064851174a923719893e932eeb2dd179..2018da72f3d98c22b40bfe03544aa94aa539fe76 > 100644 > --- a/test/mjsunit/string-indexof.js > +++ b/test/mjsunit/string-indexof.js > @@ -120,27 +120,13 @@ var lipsum = "lorem ipsum per se esse fugiendum. > itaque aiunt hanc quasi " > + "ullam corporis suscipit laboriosam, nisi ut aliquid ex ea commodi " > + "consequatur? quis autem vel eum iure reprehenderit, qui in ea " > + "voluptate velit esse, quam nihil molestiae consequatur, vel illum, " > - + "qui dolorem eum fugiat, quo voluptas nulla pariatur?\n" > - + "At vero eos et accusamus et iusto odio dignissimos ducimus, qui " > - + "blanditiis praesentium voluptatum deleniti atque corrupti, quos " > - + "dolores et quas molestias excepturi sint, obcaecati cupiditate " > - + "non provident, similique sunt in culpa, qui officia deserunt " > - + "mollitia animi, id est laborum et dolorum fuga. et harum quidem " > - + "rerum facilis est et expedita distinctio. nam libero tempore, " > - + "cum soluta nobis est eligendi optio, cumque nihil impedit, quo " > - + "minus id, quod maxime placeat, facere possimus, omnis voluptas " > - + "assumenda est, omnis dolor repellendus. temporibus autem " > - + "quibusdam et aut officiis debitis aut rerum necessitatibus " > - + "saepe eveniet, ut et voluptates repudiandae sint et molestiae " > - + "non recusandae. itaque earum rerum hic tenetur a sapiente " > - + "delectus, ut aut reiciendis voluptatibus maiores alias consequatur " > - + "aut perferendis doloribus asperiores repellat."; > + + "qui dolorem eum fugiat, quo voluptas nulla pariatur?\n"; > > assertEquals(893, lipsum.indexOf("lorem ipsum, quia dolor sit, amet"), > "Lipsum"); > // test a lot of substrings of differing length and start-position. > -for(var i = 255; i < lipsum.length; i += 3) { > - for(var len = 661; i + len < lipsum.length; len += 4) { > +for(var i = 0; i < lipsum.length; i += 3) { > + for(var len = 1; i + len < lipsum.length; len += 7) { > var substring = lipsum.substring(i, i + len); > var index = -1; > do { > > >
--~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
