Add virtual destructors for CBLSWrapper and CBLSLazyWrapper (#4298)

rkarthik2k21 · PastaPastaPasta · UdjinM6 · web-flow · commit 13cb92e27f50 · 2021-07-28T23:13:26.000+03:00
* Fixes issue #3759 Add virtual destructors for CBLSWrapper and CBLSLazyWrapper * Fix linter errors Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com> Co-authored-by: PastaPastaPasta <6443210+PastaPastaPasta@users.noreply.github.com> Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>
diff --git a/src/bls/bls.h b/src/bls/bls.h
@@ -80,6 +80,8 @@ class CBLSWrapper
         return *this;
     }
 
+    virtual ~CBLSWrapper() {}
+
     bool operator==(const C& r) const
     {
         return fValid == r.fValid && impl == r.impl;
@@ -321,6 +323,7 @@ class CBLSLazyWrapper
     {
         *this = r;
     }
+    virtual ~CBLSLazyWrapper() {}
 
     CBLSLazyWrapper& operator=(const CBLSLazyWrapper& r)
     {

Original file line number	Diff line number	Diff line change
`@@ -80,6 +80,8 @@ class CBLSWrapper`
`80`	`80`	`return *this;`
`81`	`81`	`}`
`82`	`82`
	`83`	`+ virtual ~CBLSWrapper() {}`
	`84`	`+`
`83`	`85`	`bool operator==(const C& r) const`
`84`	`86`	`{`
`85`	`87`	`return fValid == r.fValid && impl == r.impl;`
`@@ -321,6 +323,7 @@ class CBLSLazyWrapper`
`321`	`323`	`{`
`322`	`324`	`*this = r;`
`323`	`325`	`}`
	`326`	`+ virtual ~CBLSLazyWrapper() {}`
`324`	`327`
`325`	`328`	`CBLSLazyWrapper& operator=(const CBLSLazyWrapper& r)`
`326`	`329`	`{`