Alternatively you can always replace bool with a custom enum. -- Vyacheslav Egorov
On Fri, Feb 18, 2011 at 5:16 PM, <[email protected]> wrote: > LGTM, nice cleanup. > > > http://codereview.chromium.org/6542017/diff/1/src/handles.cc > File src/handles.cc (right): > > http://codereview.chromium.org/6542017/diff/1/src/handles.cc#newcode859 > src/handles.cc:859: return CompileLazyFunction(function, flag, false /* > in loop */); > The C style comment is OK, but I don't think we have any examples of the > style in the code already: > > You could also consider: > > const kInLoop = false; > return CompileLazyFunction(function, flag, kInLoop); > > but it's not obviously better. > > http://codereview.chromium.org/6542017/ > > -- > v8-dev mailing list > [email protected] > http://groups.google.com/group/v8-dev > -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
