Skip to content

Commit 8f423b8

Browse files
committed
src: remove OnScopeLeaveImpl's move assignment overload
... as it's not valid implementation and also has not been used PR-URL: nodejs/node#48732 Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent 6526326 commit 8f423b8

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

graal-nodejs/src/util.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -704,12 +704,6 @@ struct OnScopeLeaveImpl {
704704
: fn_(std::move(other.fn_)), active_(other.active_) {
705705
other.active_ = false;
706706
}
707-
OnScopeLeaveImpl& operator=(OnScopeLeaveImpl&& other) {
708-
if (this == &other) return *this;
709-
this->~OnScopeLeave();
710-
new (this)OnScopeLeaveImpl(std::move(other));
711-
return *this;
712-
}
713707
};
714708

715709
// Run a function when exiting the current scope. Used like this:

0 commit comments

Comments
 (0)