On 2014/05/08 14:52:40, Yang wrote:
On 2014/05/08 14:50:46, aandrey wrote:
> On 2014/05/08 14:32:30, Yang wrote:
> > On 2014/05/08 14:17:00, aandrey wrote:
> > > ptal
> >
> > LGTM. Though it's not clear why you are reducing the length further.
>
> I was thinking to drop the function body all together, i.e. "function
foo(arg1,
> arg2, arg3) { ...<omitted>... }", since I can't imagine how this is helpful
in
> error messages.
> IMO 128 limit is more than enough to include function name, its arguments
and
> some code, and saves some clattering comparing to 256.
Lgtm.

Ill land this the next chance I get

https://codereview.chromium.org/271733005/

--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to