Your weak callback (handle_weak) is empty and does not follow the contract.
Put object.Dispose(); there.
--
Vyacheslav Egorov
On Mon, Dec 26, 2011 at 12:55 PM, D C <thatway...@gmail.com> wrote:
> Hi:
> Thanks for you replay, I modify my source according to your words,
> It seems not work.
> here is my source snippet:
>
> void javascript_ctx_impl::call_obj_func (v8::persistent<object>
> object, const char* method, int argc, handle<value> argv[])
> {
> handlescope handle_scope;
>
> local<value> cb = object->get(string::new(method));
>
> if (!cb->isfunction()) {
> std::cerr << "method = " << method << std::endl;
> return;
> }
>
> local<function> do_action = local<function>::cast(cb);
>
> trycatch try_catch;
> /*********** ASSERT HERE ******************/
> /*********** ASSERT HERE ******************/
> /*********** ASSERT HERE ******************/
> do_action->call(object, argc, argv);
> /*********** ASSERT HERE ******************/
> /*********** ASSERT HERE ******************/
> /*********** ASSERT HERE ******************/
> if (try_catch.hascaught()) {
> v8::local<v8::message> msg = try_catch.message ();
> if (!msg->getscriptresourcename().isempty() && !msg-
> >getscriptresourcename()->isundefined())
> {
> v8::string::asciivalue name (msg-
> >getscriptresourcename());
> std::cerr << *name << std::endl;
> }
> else {
> std::cerr << "call_obj_func: runtime error." << std::endl;
> }
> }
> }
>
> template <typename T>
> class write_handle : public handle_impl_base
> {
> public:
> write_handle (boost::asio::io_service& io,
> v8::Persistent<Object>
> local,
> v8::Persistent<Object> h
> )
> : handle_impl_base (io), handle_ (h), session_ (local)
> {
> }
> public:
> void operator () (const boost::system::error_code& ec,
> std::size_t bytes_transferred)
> {
> HandleScope handle_scope;
> if (!ec) {
> Handle<Value> args[3] = {
> js::instance ().safe_new_value (session_),
> js::instance ().safe_new_value
> ("TRUE"),
> js::instance ().safe_new_value
> (bytes_transferred)
> };
>
> js::instance ().call_obj_func (handle_,
> "onHandle", 3, args);
> }
> else {
> Handle<Value> args[3] = {
> js::instance ().safe_new_value (session_),
> js::instance ().safe_new_value
> ("FALSE"),
> js::instance ().safe_new_value
> (bytes_transferred)
> };
> js::instance ().call_obj_func (handle_,
> "onHandle", 3, args);
> }
> handle_.Dispose (); session_.Dispose ();
> }
> static void handle_weak (Persistent<Value> object, void*
> parameter)
> {
> }
> private:
> v8::Persistent<Object> handle_;
> v8::Persistent<Object> session_;
> };
>
> v8::Handle<v8::Value> js_asio_socket_ip_tcp_function::async_write
> (const v8::Arguments& args)
> {
> HandleScope hScope;
> js_asio_socket_ip_tcp_function* native_obj =
> unwrap<js_asio_socket_ip_tcp_function>(args.This());
>
> if (args.Length () < 4) {
> return ThrowException (Exception::TypeError(String::New(
> "async_resolve need 4 parameters."))
> );
> }
> /** Argument check here */
> js_stream_function* s = unwrap<js_stream_function> (args[1]-
> >ToObject ());
> if (s == NULL) {
> return ThrowException (Exception::TypeError(String::New(
> "async_resolve parameter 2 error."))
> );
> }
>
> v8::Local<v8::Object> p0 = args[0]->ToObject ();
> v8::Local<v8::Integer> p2 = args[2]->ToUint32 ();
>
> v8::Persistent<Object> handle;
> v8::Persistent<Object> sessin;
>
> if (args[3]->ToObject ()->IsFunction ()) {
> v8::Local<v8::Function> f = v8::Local<v8::Function>::Cast(args
> [3]->ToObject());
> handle = v8::Persistent<v8::Object>::New(f);
> }
> else {
> handle = v8::Persistent<Object>::New (args[3]->ToObject ());
> }
>
> handle.MakeWeak (NULL, write_handle<void>::handle_weak);
> handle.MarkIndependent ();
>
> sessin = v8::Persistent<Object>::New (p0);
> boost::asio::async_write (*(native_obj->socket_),
> boost::asio::buffer (s->get (), p2->Value ()),
> boost::asio::transfer_all (),
> make_concrete_handle (write_handle <void> (native_obj-
> >socket_->get_io_service (), sessin, handle)
> )
> );
> return v8::Undefined ();
> }
>
> if you have some suggestion, it will be very thankful.
>
>
> On 12月26日, 午前8:15, Vyacheslav Egorov <vego...@chromium.org> wrote:
> > Every weak callback should either revive (via ClearWeak or MakeWeak) or
> > destroy (via Dispose) the handle for which it was called. This contract
> is
> > described in v8.h, see the comment above WeakReferenceCallback
> definition.
> >
> > You have a callback that does not follow this contract.
> >
> > --
> > Vyacheslav Egorov
> > On Dec 25, 2011 3:41 PM, "D C" <thatway...@gmail.com> wrote:
> >
> >
> >
> >
> >
> >
> >
> > > Hi all:
> > > I stuck here for about two weeks.
> > > Here is my stack trace.
> > >
> =========================================================================
> > > Agent.exe!v8::internal::OS::DebugBreak() Line 930 C++
> > > Agent.exe!v8::internal::OS::Abort() Line 925 C++
> > > Agent.exe!V8_Fatal(const char * file=0x00d8fcd8, int line=237,
> const
> > > char * format=0x00d7bd18, ...) Line 59 C++
> > > Agent.exe!CheckHelper(const char * file=0x00d8fcd8, int
> line=237,
> > > const char * source=0x00d8fe34, bool condition=false) Line 60 + 0x16
> > > bytes C++
> >
> > >
> Agent.exe!v8::internal::GlobalHandles::Node::PostGarbageCollectionProcessin
> g(v8::internal::Isolate
> > > * isolate=0x014b0068, v8::internal::GlobalHandles *
> > > global_handles=0x003ef478) Line 237 + 0x27 bytes C++
> > > Agent.exe!
> >
> > >
> v8::internal::GlobalHandles::PostGarbageCollectionProcessing(v8::internal::
> GarbageCollector
> > > collector=SCAVENGER) Line 540 + 0x12 bytes C++
> > > Agent.exe!
> > >
> v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector
> > > collector=SCAVENGER, v8::internal::GCTracer * tracer=0x0012e0b4) Line
> > > 822 + 0x15 bytes C++
> > > Agent.exe!
> > > v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace
> > > space=NEW_SPACE, v8::internal::GarbageCollector collector=SCAVENGER)
> > > Line 518 + 0x16 bytes C++
> > > Agent.exe!
> > > v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace
> > > space=NEW_SPACE) Line 443 C++
> > > Agent.exe!
> > > v8::internal::Factory::NewStringFromUtf8(v8::internal::Vector<char
> > > const > string={...}, v8::internal::PretenureFlag
> > > pretenure=NOT_TENURED) Line 186 + 0xea bytes C++
> > > Agent.exe!v8::String::New(const char * data=0x00d68254, int
> > > length=8) Line 4410 C++
> > >
> Agent.exe!javascript_ctx_impl::call_obj_func(v8::Handle<v8::Object>
> > > object={...}, const char * method=0x00d68254, int argc=3,
> > > v8::Handle<v8::Value> * argv=0x0012e4b0) Line 111 + 0x12 bytes C++
> > > Agent.exe!read_handle<void>::operator()(const
> > > boost::system::error_code & ec={...}, unsigned int
> > > bytes_transferred=661) Line 226 C++
> > >
> =========================================================================
> > > It seem like each time GC, String::New ASSERT, who can help me about
> > > that.
> >
> > > --
> > > v8-users mailing list
> > > v8-users@googlegroups.com
> > >http://groups.google.com/group/v8-users
>
> --
> v8-users mailing list
> v8-users@googlegroups.com
> http://groups.google.com/group/v8-users
>
--
v8-users mailing list
v8-users@googlegroups.com
http://groups.google.com/group/v8-users
