proposal-signals icon indicating copy to clipboard operation
proposal-signals copied to clipboard

"Effect" is undefined in `Computed.get`

Open prophile opened this issue 1 year ago • 2 comments

The algorithm specification for "Method: Signal.Computed.prototype.get" says this in phase 1:

If the current execution context is notifying or if this Signal has the state ~computing~, or if this signal is an Effect and computing a computed Signal, throw an exception.

There's no other reference to an "Effect" so it's not clear what this means. There are some references to "effect Signal" elsewhere but this is also not defined anywhere.

prophile avatar Apr 29 '24 17:04 prophile

Good catch @prophile! I think this is some language left over from an earlier draft of the proposal. @littledan Should this just be updated to say Watcher or is there a more involved change here?

EisenbergEffect avatar Apr 29 '24 17:04 EisenbergEffect

Yes, we should just be referring to Watcher here.

littledan avatar May 13 '24 20:05 littledan