Reviewers: Søren Gjesse,

Description:
Change locationFromPosition() and locationFromLine() to use a binary
search to locate line numbers from position numbers.  Modify test
debug-sourceinfo.js to include more tests, including error conditions.

Ran the test successfully with the old and new version of messages.js.
This implies things work the same, including "error" conditions of
supplying bad position numbers (that is, querying "<0" returns 0 while
querying ">max" returns null).

Please review this at http://codereview.chromium.org/118371

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
   M     src/messages.js
   M     test/mjsunit/debug-sourceinfo.js



--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to