2010-07-29 Michael Saboff <msaboff@apple.com>
authorcommit-queue <commit-queue@webkit.org>
Thu, 29 Jul 2010 21:33:19 +0000 (21:33 +0000)
committerAdemar de Souza Reis Jr <ademar.reis@openbossa.org>
Mon, 6 Dec 2010 22:05:59 +0000 (19:05 -0300)
commitdf5e09037d1c309af3ad6ab745b9013e7aeebb5b
treed7152df36bb8c3a7da9b59e65c0ac0ebde9c008d
parente7c83923578b4d90e193a95d01d6987bb02fcac3
2010-07-29  Michael Saboff  <msaboff@apple.com>

        Reviewed by Oliver Hunt.

        Fixed issue where RegExp greedy jit code loops when no input is
        consumed.  Changed the code to only loop if some input was consumed,
        but fall through if we successfully match an alternative that
        doesn't consume any input.
        https://bugs.webkit.org/show_bug.cgi?id=42664

        * yarr/RegexJIT.cpp:
        (JSC::Yarr::RegexGenerator::generateParenthesesGreedyNoBacktrack):
2010-07-29  Michael Saboff  <msaboff@apple.com>

        Reviewed by Oliver Hunt.

        Added tests to correspond to the changes made in
        JavaScriptCore/yarr/RegexJIT.cpp to fix
        https://bugs.webkit.org/show_bug.cgi?id=42664.
        Note that some of the new test cases fail due to one or more
        unrelated bugs in the pcre interpreter.  The expected results
        for these tests will need to be updated when the implementation
        conforms to the JS standard.

        * fast/js/regexp-look-ahead-empty-expected.txt: Added.
        * fast/js/regexp-look-ahead-empty.html: Added.
        * fast/js/script-tests/regexp-look-ahead-empty.js: Added.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@64307 268f45cc-cd09-0410-ab3c-d52691b4dbfc
JavaScriptCore/ChangeLog
JavaScriptCore/yarr/RegexJIT.cpp
LayoutTests/ChangeLog
LayoutTests/fast/js/regexp-look-ahead-empty-expected.txt [new file with mode: 0644]
LayoutTests/fast/js/regexp-look-ahead-empty.html [new file with mode: 0644]
LayoutTests/fast/js/script-tests/regexp-look-ahead-empty.js [new file with mode: 0644]