[v8-dev] Re: Light-weight External objects

2009-03-22 Thread Ivan Posva
On Sun, Mar 22, 2009 at 18:43, Christian Plesner Hansen wrote: > > Ivan, > The way this is expressed in the api it effectively removes the > distinction between externals and integers, which I would say is a Bad > Thing(TM).  As the code is now it no longer holds that > External::Cast(x) succeeds

[v8-dev] Issue 282 in v8: Multiple crashes in test-api under debug gc-greedy

2009-03-22 Thread codesite-noreply
Updates: Summary: Multiple crashes in test-api under debug gc-greedy Comment #1 on issue 282 by christian.plesner.hansen: Multiple crashes in test-api under debug gc-greedy http://code.google.com/p/v8/issues/detail?id=282 Crashes also occur in PropertyHandler, DescriptorInheritance,

[v8-dev] Issue 282 in v8: PropertyHandler test in test-api crashes under debug gc-greedy

2009-03-22 Thread codesite-noreply
Status: Accepted Owner: Labels: Type-Bug Priority-Medium New issue 282 by christian.plesner.hansen: PropertyHandler test in test-api crashes under debug gc-greedy http://code.google.com/p/v8/issues/detail?id=282 Reason: KERN_INVALID_ADDRESS at address: 0x0beefdac 0x001cb43c in v8::interna

[v8-dev] [v8 commit] r1570 - Fixed windows build.

2009-03-22 Thread codesite-noreply
Author: christian.plesner.hansen Date: Sun Mar 22 19:25:40 2009 New Revision: 1570 Modified: branches/bleeding_edge/test/cctest/test-api.cc Log: Fixed windows build. Review URL: http://codereview.chromium.org/52004 Modified: branches/bleeding_edge/test/cctest/test-api.cc ==

[v8-dev] Fixed windows build.

2009-03-22 Thread christian . plesner . hansen
Reviewers: iposva, Message: Post facto review - I'm submitting the code right away. Description: Fixed windows build. Please review this at http://codereview.chromium.org/52004 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M test/cctest/test-api.cc I

[v8-dev] Re: Light-weight External objects

2009-03-22 Thread Christian Plesner Hansen
Ivan, The way this is expressed in the api it effectively removes the distinction between externals and integers, which I would say is a Bad Thing(TM). As the code is now it no longer holds that External::Cast(x) succeeds iff x->IsExternal(). This may just be an oversight but if we change IsExte

[v8-dev] Re: Made regexp robust against changes to a string's implementation.

2009-03-22 Thread Christian Plesner Hansen
This change seems to have caused a fairly big performance regression on arm. Before this change the longest running test took just over a minute, after this change 48 tests time out because they take over two minutes. On Sat, Mar 21, 2009 at 12:26 AM, wrote: > > Addressed issues except: > > >