From cc353244c25e38bdb344023f0bdedfd4c40a4d54 Mon Sep 17 00:00:00 2001 From: Grzegorz Caban Date: Wed, 28 Aug 2024 09:30:02 +0100 Subject: [PATCH] updating outdated comment on defineProperty --- .../main/java/org/mozilla/javascript/ScriptableObject.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/rhino/src/main/java/org/mozilla/javascript/ScriptableObject.java b/rhino/src/main/java/org/mozilla/javascript/ScriptableObject.java index f97873dd12..c871c56b07 100644 --- a/rhino/src/main/java/org/mozilla/javascript/ScriptableObject.java +++ b/rhino/src/main/java/org/mozilla/javascript/ScriptableObject.java @@ -1698,9 +1698,8 @@ public void defineProperty( * need to have access to target object instance, this allows for defining properties on * LambdaConstructor prototype providing getter and setter logic with java instance methods. If * a property with the same name already exists, then it will be replaced. This property will - * appear to the JavaScript user exactly like any other property -- unlike Function properties - * and those based on reflection, the property descriptor will only reflect the value as defined - * by this function. + * appear to the JavaScript user exactly like descriptor with a getter and setter, just as if + * they had been defined in JavaScript using Object.defineOwnProperty. * * @param name the name of the property * @param getter a function that given Scriptable `this` returns the value of the property. If