Fixed Op destructors not being called

pull/3341/head
murgatroid99 9 years ago
parent e8659971a9
commit 474f53b138
  1. 5
      src/node/ext/call.cc
  2. 2
      src/node/ext/call.h

@ -172,6 +172,9 @@ Handle<Value> Op::GetOpType() const {
return NanEscapeScope(NanNew<String>(GetTypeString())); return NanEscapeScope(NanNew<String>(GetTypeString()));
} }
Op::~Op() {
}
class SendMetadataOp : public Op { class SendMetadataOp : public Op {
public: public:
Handle<Value> GetNodeValue() const { Handle<Value> GetNodeValue() const {
@ -325,7 +328,7 @@ class ReadMessageOp : public Op {
} }
~ReadMessageOp() { ~ReadMessageOp() {
if (recv_message != NULL) { if (recv_message != NULL) {
gpr_free(recv_message); grpc_byte_buffer_destroy(recv_message);
} }
} }
Handle<Value> GetNodeValue() const { Handle<Value> GetNodeValue() const {

@ -88,6 +88,7 @@ struct Resources {
class Op { class Op {
public: public:
virtual ~Op();
virtual v8::Handle<v8::Value> GetNodeValue() const = 0; virtual v8::Handle<v8::Value> GetNodeValue() const = 0;
virtual bool ParseOp(v8::Handle<v8::Value> value, grpc_op *out, virtual bool ParseOp(v8::Handle<v8::Value> value, grpc_op *out,
shared_ptr<Resources> resources) = 0; shared_ptr<Resources> resources) = 0;
@ -98,7 +99,6 @@ class Op {
}; };
typedef std::vector<unique_ptr<Op>> OpVec; typedef std::vector<unique_ptr<Op>> OpVec;
struct tag { struct tag {
tag(NanCallback *callback, OpVec *ops, tag(NanCallback *callback, OpVec *ops,
shared_ptr<Resources> resources); shared_ptr<Resources> resources);

Loading…
Cancel
Save