src: add not-weak DCHECK to PersistentToLocal::Strong · nodejs/node@db4b192 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 633
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit db4b192

Browse filesBrowse the repository at this point in the historyBrowse files

addaleax

authored and

targos

committed

src: add not-weak DCHECK to PersistentToLocal::Strong

Refs: #38821 (comment) PR-URL: #38875 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Bryan English <bryan@bryanenglish.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>

1 parent 7d35fa7 commit db4b192
Copy full SHA for db4b192

1 file changed

+1Lines changed: 1 addition & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• src
  • util.h

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎src/util.h‎

Copy file name to clipboardExpand all lines: src/util.h

+1Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -772,6 +772,7 @@ class PersistentToLocal {
772	772	template <class TypeName>
773	773	static inline v8::Local<TypeName> Strong(
774	774	const v8::PersistentBase<TypeName>& persistent) {
	775	+ DCHECK(!persistent.IsWeak());
775	776	return *reinterpret_cast<v8::Local<TypeName>*>(
776	777	const_cast<v8::PersistentBase<TypeName>*>(&persistent));
777	778	}

0 commit comments

Comments

0 (0)