Open
Description
In Wren, assignment returns its value, like in C:
var a
System.print(a = 5) // 5When implementing setters, we need to maintain this behavior:
// Not good
class C {
static a=(value) {
__a = value
}
}
System.print(C.a = 5) // null
// Good
class C {
static a=(value) { __a = value }
}
System.print(C.a = 5) // 5
// Also good
class C {
static a=(value) {
__a = value
return value
}
}
System.print(C.a = 5) // 5Generally, the CLI do that, except in one foreign setter - Stdin.isRaw=(_):
Lines 509 to 525 in b82cf5a
Instead we should do:
void stdinIsRawSet(WrenVM* vm)
{
initStdin();
isStdinRaw = wrenGetSlotBool(vm, 1);
if (uv_guess_handle(stdinDescriptor) == UV_TTY)
{
uv_tty_t* handle = (uv_tty_t*)stdinStream;
uv_tty_set_mode(handle, isStdinRaw ? UV_TTY_MODE_RAW : UV_TTY_MODE_NORMAL);
}
else
{
// Can't set raw mode when not talking to a TTY.
// TODO: Make this a runtime error?
}
wrenEnsureSlots(vm, 1); // Is that required? Probably not
wrenSetSlotBool(vm, 0, isStdinRaw);
}Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Milestone
Relationships
Participants
Activity
wrenCopySlot()wren-lang/wren#836joshgoebel commentedon Apr 25, 2021
While the other issue is being discussed (adding the API) would you like to go ahead and make a PR for this? It seems the agreement was reached that this is the correct behavior at least. :-)
(chore) consistent setter behavior
aosenkidu commentedon Feb 24, 2023
Interesting point of view. As we usually do not really "use" setters, but what other languages call "property access".
Aka, we pretend there is no setter and we could write to the field/attribute directly:
obj.property = valuehence we have the idea from C etc. that:
var x = obj.property = valueassigns value to the variabel x after calling the setter property on the object obj.
In my opinion the compiler should take care on the level of that expression that this is the case. And there is no reason that the setter is returning its argument.
In fact the setter should return "this". And that also should be the responsibility of the compiler. So you can write:
obj.propertyOne(value).propertyTwo(anotherValue)Albeit, while I'm typing this: I wonder if you can call a setter in wren, or if a setter can only be accessed via "="?