[webkit-reviews] review denied: [Bug 6252] JavaScript 1.6 Array.lastIndexOf : [Attachment 6777] Updated lastIndexOf implementation

[bugzilla-request-daemon at opendarwin.org]

Darin Adler <darin at apple.com> has denied Darin Adler <darin at apple.com>'s
request for review:
Bug 6252: JavaScript 1.6 Array.lastIndexOf
http://bugzilla.opendarwin.org/show_bug.cgi?id=6252

Attachment 6777: Updated lastIndexOf implementation
http://bugzilla.opendarwin.org/attachment.cgi?id=6777&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
This code looks like it won't work in a couple of ways:

    1) If 0 is passed it looks like it will search the entire array, but that's
not what the documentation seems to call for.

    2) The for loop seems to stop with index == 1, so it will miss index == 0.

In any case, we need a test case and change log to land a change like this.
Please add a test case that covers both of these issues and test the code
before putting it up for review again.