diff options
author | GauthamBanasandra <gautham.bangalore@gmail.com> | 2019-07-18 00:11:38 +0530 |
---|---|---|
committer | Michaƫl Zasso <targos@protonmail.com> | 2019-07-22 21:20:42 +0200 |
commit | 74243da70792f49cabd5e2b8bff5b915b4a3e759 (patch) | |
tree | 9ca3685f0d926a0528c34046afd5a14ec164ec3a | |
parent | 7b0b06d735c90c41900efe5948a0aee6c0a745df (diff) | |
download | android-node-v8-74243da70792f49cabd5e2b8bff5b915b4a3e759.tar.gz android-node-v8-74243da70792f49cabd5e2b8bff5b915b4a3e759.tar.bz2 android-node-v8-74243da70792f49cabd5e2b8bff5b915b4a3e759.zip |
src: add public virtual destructor for KVStore
As KVStore has derived classes, it is essential to
declare a public virtual destructor in the base
KVStore class. Otherwise, deleting derived class
instances using base class pointers would
potentially cause undefined behaviour.
Additionally, since we are implementing a non-default
destructor, the special member functions have also
been implemented in order to abide by the rule of five.
PR-URL: https://github.com/nodejs/node/pull/28737
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
-rw-r--r-- | src/env.h | 7 |
1 files changed, 7 insertions, 0 deletions
@@ -582,6 +582,13 @@ class AsyncRequest : public MemoryRetainer { class KVStore { public: + KVStore() = default; + virtual ~KVStore() = default; + KVStore(const KVStore&) = delete; + KVStore& operator=(const KVStore&) = delete; + KVStore(KVStore&&) = delete; + KVStore& operator=(KVStore&&) = delete; + virtual v8::Local<v8::String> Get(v8::Isolate* isolate, v8::Local<v8::String> key) const = 0; virtual void Set(v8::Isolate* isolate, |