nodejs
diff --git a/Collapse file
‎src/node_object_wrap.h‎
Copy file name to clipboardExpand all lines: src/node_object_wrap.h
+2Lines changed: 2 additions & 0 deletions b/Collapse file
‎src/node_object_wrap.h‎
Copy file name to clipboardExpand all lines: src/node_object_wrap.h
+2Lines changed: 2 additions & 0 deletions
diff --git a/Collapse file
‎tools/cpplint.py‎
Copy file name to clipboardExpand all lines: tools/cpplint.py
+26Lines changed: 26 additions & 0 deletions b/Collapse file
‎tools/cpplint.py‎
Copy file name to clipboardExpand all lines: tools/cpplint.py
+26Lines changed: 26 additions & 0 deletions
@@ -65,6 +65,7 @@ class ObjectWrap {
   }
 
 
+  // NOLINTNEXTLINE(runtime/v8_persistent)
   inline v8::Persistent<v8::Object>& persistent() {
     return handle_;
   }
@@ -122,6 +123,7 @@ class ObjectWrap {
     delete wrap;
   }
 
+  // NOLINTNEXTLINE(runtime/v8_persistent)
   v8::Persistent<v8::Object> handle_;
 };
 
 
@@ -321,6 +321,7 @@
     'runtime/string',
     'runtime/threadsafe_fn',
     'runtime/vlog',
+    'runtime/v8_persistent',
     'whitespace/blank_line',
     'whitespace/braces',
     'whitespace/comma',
@@ -627,6 +628,8 @@
 
 _NULL_TOKEN_PATTERN = re.compile(r'\bNULL\b')
 
+_V8_PERSISTENT_PATTERN = re.compile(r'\bv8::Persistent\b')
+
 _RIGHT_LEANING_POINTER_PATTERN = re.compile(r'[^=|(,\s><);&?:}]'
                                             r'(?<!(sizeof|return))'
                                             r'\s\*[a-zA-Z_][0-9a-zA-Z_]*')
@@ -4547,6 +4550,28 @@ def CheckNullTokens(filename, clean_lines, linenum, error):
     error(filename, linenum, 'readability/null_usage', 2,
           'Use nullptr instead of NULL')
 
+def CheckV8PersistentTokens(filename, clean_lines, linenum, error):
+  """Check v8::Persistent usage.
+
+  Args:
+    filename: The name of the current file.
+    clean_lines: A CleansedLines instance containing the file.
+    linenum: The number of the line to check.
+    error: The function to call with any errors found.
+  """
+  line = clean_lines.elided[linenum]
+
+  # Avoid preprocessor lines
+  if Match(r'^\s*#', line):
+    return
+
+  if line.find('/*') >= 0 or line.find('*/') >= 0:
+    return
+
+  for match in _V8_PERSISTENT_PATTERN.finditer(line):
+    error(filename, linenum, 'runtime/v8_persistent', 2,
+          'Use v8::Global instead of v8::Persistent')
+
 def CheckLeftLeaningPointer(filename, clean_lines, linenum, error):
   """Check for left-leaning pointer placement.
 
@@ -4723,6 +4748,7 @@ def CheckStyle(filename, clean_lines, linenum, file_extension, nesting_state,
   CheckCheck(filename, clean_lines, linenum, error)
   CheckAltTokens(filename, clean_lines, linenum, error)
   CheckNullTokens(filename, clean_lines, linenum, error)
+  CheckV8PersistentTokens(filename, clean_lines, linenum, error)
   CheckLeftLeaningPointer(filename, clean_lines, linenum, error)
   classinfo = nesting_state.InnermostClass()
   if classinfo:
Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,7 @@ class ObjectWrap {`
`65`	`65`	`}`
`66`	`66`
`67`	`67`
	`68`	`+ // NOLINTNEXTLINE(runtime/v8_persistent)`
`68`	`69`	`inline v8::Persistent<v8::Object>& persistent() {`
`69`	`70`	`return handle_;`
`70`	`71`	`}`
`@@ -122,6 +123,7 @@ class ObjectWrap {`
`122`	`123`	`delete wrap;`
`123`	`124`	`}`
`124`	`125`
	`126`	`+ // NOLINTNEXTLINE(runtime/v8_persistent)`
`125`	`127`	`v8::Persistent<v8::Object> handle_;`
`126`	`128`	`};`
`127`	`129`