Skip to content

Commit 5bf9435

Browse files
danbevjasnell
authored andcommitted
src: remove unused StringValue macro parameters
Currently, there are a few places where macro functions passed to the PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES and PER_ISOLATE_STRING_PROPERTIES macros, don't use the StringValue parameter. This commit removes the StringValue parameter where it is not used. PR-URL: #7905 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
1 parent 8fa981b commit 5bf9435

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

src/env-inl.h

+6-6
Original file line numberDiff line numberDiff line change
@@ -443,9 +443,9 @@ inline v8::Local<v8::Object> Environment::NewInternalFieldObject() {
443443
return m_obj.ToLocalChecked();
444444
}
445445

446-
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName, StringValue)
447-
#define VS(PropertyName, StringValue) V(v8::String, PropertyName, StringValue)
448-
#define V(TypeName, PropertyName, StringValue) \
446+
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName)
447+
#define VS(PropertyName, StringValue) V(v8::String, PropertyName)
448+
#define V(TypeName, PropertyName) \
449449
inline \
450450
v8::Local<TypeName> IsolateData::PropertyName(v8::Isolate* isolate) const { \
451451
/* Strings are immutable so casting away const-ness here is okay. */ \
@@ -457,9 +457,9 @@ inline v8::Local<v8::Object> Environment::NewInternalFieldObject() {
457457
#undef VS
458458
#undef VP
459459

460-
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName, StringValue)
461-
#define VS(PropertyName, StringValue) V(v8::String, PropertyName, StringValue)
462-
#define V(TypeName, PropertyName, StringValue) \
460+
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName)
461+
#define VS(PropertyName, StringValue) V(v8::String, PropertyName)
462+
#define V(TypeName, PropertyName) \
463463
inline v8::Local<TypeName> Environment::PropertyName() const { \
464464
return isolate_data()->PropertyName(isolate()); \
465465
}

src/env.h

+9-9
Original file line numberDiff line numberDiff line change
@@ -275,9 +275,9 @@ class IsolateData {
275275
inline uv_loop_t* event_loop() const;
276276
inline uint32_t* zero_fill_field() const;
277277

278-
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName, StringValue)
279-
#define VS(PropertyName, StringValue) V(v8::String, PropertyName, StringValue)
280-
#define V(TypeName, PropertyName, StringValue) \
278+
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName)
279+
#define VS(PropertyName, StringValue) V(v8::String, PropertyName)
280+
#define V(TypeName, PropertyName) \
281281
inline v8::Local<TypeName> PropertyName(v8::Isolate* isolate) const;
282282
PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES(VP)
283283
PER_ISOLATE_STRING_PROPERTIES(VS)
@@ -286,9 +286,9 @@ class IsolateData {
286286
#undef VP
287287

288288
private:
289-
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName, StringValue)
290-
#define VS(PropertyName, StringValue) V(v8::String, PropertyName, StringValue)
291-
#define V(TypeName, PropertyName, StringValue) \
289+
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName)
290+
#define VS(PropertyName, StringValue) V(v8::String, PropertyName)
291+
#define V(TypeName, PropertyName) \
292292
v8::Eternal<TypeName> PropertyName ## _;
293293
PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES(VP)
294294
PER_ISOLATE_STRING_PROPERTIES(VS)
@@ -505,9 +505,9 @@ class Environment {
505505

506506
// Strings and private symbols are shared across shared contexts
507507
// The getters simply proxy to the per-isolate primitive.
508-
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName, StringValue)
509-
#define VS(PropertyName, StringValue) V(v8::String, PropertyName, StringValue)
510-
#define V(TypeName, PropertyName, StringValue) \
508+
#define VP(PropertyName, StringValue) V(v8::Private, PropertyName)
509+
#define VS(PropertyName, StringValue) V(v8::String, PropertyName)
510+
#define V(TypeName, PropertyName) \
511511
inline v8::Local<TypeName> PropertyName() const;
512512
PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES(VP)
513513
PER_ISOLATE_STRING_PROPERTIES(VS)

0 commit comments

Comments
 (0)