Question

我正在努力更新node-mapserver，将mapserver分支中的class-refactor库包装为更易于维护的代码结构（基于node-ogr组织的更改）

到目前为止，我已经重构了主模块接口并启动了第一个要包装的类，一个名为errorObj的mapserver C结构，我用class MSError包装。扩展构建但我面临的是第一次触发新的MSError对象来包装errorObj时扩展中的运行时断言错误。

Fatal error in ../deps/v8/src/api.h, line 297
CHECK(allow_empty_handle || that != __null) failed

对于节点0.10，此后有一个非常冗长的堆栈跟踪，可以在travis build 27.2中看到。我在堆栈跟踪中找不到任何特别有用的东西。

扩展构建和一些测试通过（从库本身导出的符号和方法），但是当库尝试创建包装C对象的Object时失败。

在javascript中，我致电mapserver.getError()。在扩展中，这将调用mapserver方法errorObj* err = msGetErrorObj();并返回return scope.Close(MSError::New(err));。 MSError的New方法执行此操作：

Handle<Value> MSError::New(errorObj *err) {
  HandleScope scope;
  MSError *wrapped = new MSError(err);
  Handle<Value> ext = External::New(wrapped);
  Handle<Object> obj = MSError::constructor->GetFunction()->NewInstance(1, &ext);
  return scope.Close(obj);
}

我试图使用gdb来调试这个，我能用我有限的技能解决这个问题的最好方法就是在这次调用时发生错误：

  Handle<Object> obj = MSError::constructor->GetFunction()->NewInstance(1, &ext);

我可以从中收集的是ext为空，这意味着External::New(wrapped)调用未返回有效值。我已经确认errorObj* err确实指向一个有效的，正确初始化的errorObj结构。为了澄清，没有实际的错误，mapserver总是返回一个有效的errorObj，但如果此时没有错误，则代码为0。

希望是相关代码。

ms_error.hpp

#ifndef __NODE_MS_ERROR_H__
#define __NODE_MS_ERROR_H__

#include <v8.h>

#include <node.h>
#include <node_object_wrap.h>

#include <mapserver.h>

using namespace v8;
using namespace node;

class MSError: public node::ObjectWrap {
  public:
    static Persistent<FunctionTemplate> constructor;
    static void Initialize(Handle<Object> target);
    static Handle<Value> New(const Arguments &args);
    static Handle<Value> New(errorObj *err);

    MSError();
    MSError(errorObj *err);
    inline errorObj *get() { return this_; }

  private:
    ~MSError();
    errorObj *this_;
};

#endif

ms_error.cpp

#include "ms_error.hpp"

Persistent<FunctionTemplate> MSError::constructor;

void MSError::Initialize(Handle<Object> target) {
  HandleScope scope;

  constructor = Persistent<FunctionTemplate>::New(FunctionTemplate::New(MSError::New));
  constructor->InstanceTemplate()->SetInternalFieldCount(1);
  constructor->SetClassName(String::NewSymbol("MSError"));

  // constructor->InstanceTemplate()->SetNamedPropertyHandler(MSError::NamedPropertyGetter, NULL, MSError::NamedPropertyQuery, NULL, MSError::NamedPropertyEnumerator);

  target->Set(String::NewSymbol("MSError"), constructor->GetFunction());
}

MSError::MSError(errorObj *err) : ObjectWrap(), this_(err) { }

MSError::MSError() : ObjectWrap(), this_(0) { }

MSError::~MSError() { }

Handle<Value> MSError::New(const Arguments& args)
{
  HandleScope scope;

  if (!args.IsConstructCall())
    return ThrowException(String::New("Cannot call constructor as function, you need to use 'new' keyword"));

  if (args[0]->IsExternal()) {
    Local<External> ext = Local<External>::Cast(args[0]);
    void *ptr = ext->Value();
    MSError *f =  static_cast<MSError *>(ptr);
    f->Wrap(args.This());
    return args.This();
  }

  return args.This();
}

Answer 1

答案是上面的代码很好。实际的错误是MSError :: constructor没有被初始化，因为我忽略了从主模块代码中调用它。实际的null对象是MSError :: constructor-＆gt; GetFunction（）。

感谢Ben Noordhuis指出我们正确的方向。

运行时失败类包装C对象

1 个答案: