http://codereview.chromium.org/1737009/diff/2001/3002
File src/liveedit.h (right):

http://codereview.chromium.org/1737009/diff/2001/3002#newcode121
src/liveedit.h:121: class Comparator {
On 2010/04/23 19:43:43, Evan Martin wrote:
Since nothing else in the header needed to change when you renamed
this, I guess
that means it is only used within liveedit.cc?  You might consider
putting this
class in the .cc file in an anonymous namespace.

I think I would make it difficult to test this class. What would you
say?

http://codereview.chromium.org/1737009/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to