Skip to content

Conversation

@rkarthik2k21
Copy link

Add virtual destructors for CBLSWrapper and CBLSLazyWrapper. This prevents from potential memory leaks when derived classes are deleted (using the base pointer)

Add virtual destructors for CBLSWrapper and CBLSLazyWrapper
@UdjinM6 UdjinM6 added this to the 18 milestone Jul 28, 2021
UdjinM6
UdjinM6 previously approved these changes Jul 28, 2021
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, utACK

@UdjinM6 UdjinM6 dismissed their stale review July 28, 2021 15:41

oh, wait... linter isn't happy

Co-authored-by: UdjinM6 <[email protected]>
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK for squash merge once CI is happy

@PastaPastaPasta PastaPastaPasta requested a review from UdjinM6 July 28, 2021 18:20
@PastaPastaPasta PastaPastaPasta changed the base branch from master to develop July 28, 2021 18:21
@PastaPastaPasta PastaPastaPasta dismissed their stale review July 28, 2021 18:21

The base branch was changed.

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-utACK for squash merge

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@UdjinM6 UdjinM6 changed the title Fixes issue #3759 Add virtual destructors for CBLSWrapper and CBLSLazyWrapper Jul 28, 2021
@UdjinM6 UdjinM6 merged commit 13cb92e into dashpay:develop Jul 28, 2021
gades pushed a commit to cosanta/cosanta-core that referenced this pull request May 11, 2022
…4298)

* Fixes issue dashpay#3759

Add virtual destructors for CBLSWrapper and CBLSLazyWrapper

* Fix linter errors

Co-authored-by: UdjinM6 <[email protected]>

Co-authored-by: PastaPastaPasta <[email protected]>
Co-authored-by: UdjinM6 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants