[gjs/mozjs91: 10/21] Remove JSPROP_GETTER - https://bugzilla.mozilla.org/show_bug.cgi?id=1713083




commit fcda54e84a5cbfbd4db74a4efd4bcea4c0df7e0a
Author: Evan Welsh <contact evanwelsh com>
Date:   Sat Jul 10 20:30:39 2021 -0700

    Remove JSPROP_GETTER - https://bugzilla.mozilla.org/show_bug.cgi?id=1713083

 gjs/jsapi-dynamic-class.cpp | 6 +-----
 modules/system.cpp          | 2 +-
 2 files changed, 2 insertions(+), 6 deletions(-)
---
diff --git a/gjs/jsapi-dynamic-class.cpp b/gjs/jsapi-dynamic-class.cpp
index bd305364..038bd0db 100644
--- a/gjs/jsapi-dynamic-class.cpp
+++ b/gjs/jsapi-dynamic-class.cpp
@@ -12,7 +12,7 @@
 #include <js/CallArgs.h>  // for JSNative
 #include <js/Class.h>
 #include <js/ComparisonOperators.h>
-#include <js/PropertyDescriptor.h>  // for JSPROP_GETTER
+#include <js/PropertyDescriptor.h>
 #include <js/Realm.h>  // for GetRealmObjectPrototype
 #include <js/RootingAPI.h>
 #include <js/TypeDecls.h>
@@ -216,10 +216,6 @@ gjs_define_property_dynamic(JSContext       *cx,
     if (!setter_obj)
         return false;
 
-    // TODO(mozjs91): See for JSPROP_GETTER removal
-    // https://bugzilla.mozilla.org/show_bug.cgi?id=1713083 flags |=
-    // JSPROP_GETTER | JSPROP_SETTER;
-
     return JS_DefineProperty(cx, proto, prop_name, getter_obj, setter_obj,
                              flags);
 }
diff --git a/modules/system.cpp b/modules/system.cpp
index 9ea3bf41..e589d606 100644
--- a/modules/system.cpp
+++ b/modules/system.cpp
@@ -254,7 +254,7 @@ gjs_js_define_system_stuff(JSContext              *context,
                                  GJS_MODULE_PROP_FLAGS | JSPROP_READONLY) &&
            JS_DefinePropertyById(context, module, gjs->atoms().program_args(),
                                  program_args_getter, nullptr,
-                                 GJS_MODULE_PROP_FLAGS | JSPROP_GETTER) &&
+                                 GJS_MODULE_PROP_FLAGS) &&
            JS_DefinePropertyById(context, module, gjs->atoms().version(),
                                  GJS_VERSION,
                                  GJS_MODULE_PROP_FLAGS | JSPROP_READONLY);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]