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.


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