Oh and please consistently use past tense: fixed. -- M
On Wed, Oct 20, 2010 at 1:25 PM, <[email protected]> wrote: > LGTM > > > http://codereview.chromium.org/3898006/diff/1/2 > File ChangeLog (right): > > http://codereview.chromium.org/3898006/diff/1/2#newcode13 > ChangeLog:13: (by Justin Schuh <[email protected]>) > I would remove the attribution line here. The Solaris platform fix > should be attributed as well if we want these. Let's just leave them > out. The code reviews contains details and so does the svn history. > > http://codereview.chromium.org/3898006/show > -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
