search

WebAudio / web-audio-api

The Web Audio API v1.0, developed by the W3C Audio WG
https://webaudio.github.io/web-audio-api/
Other
1.04k stars 166 forks source link

AudioNode.connect method says output param is optional but input is not. #2319

Open rtoy opened 3 years ago

rtoy commented 3 years ago

Describe the issue

In the parameter table for AudioNdoe.connect, the list of parameters says output is optional, but input is not. That seems not right given the IDL for connect. Both are optional.

Also, the table for the connect method for an AudioParam says both args are optional. The destinationParam argument is not optional.

AudioNode.disconnect with 3 args has the same problem as connect: the output is optional, but the input is not.

Perhaps these are all confusions on bikeshed's part? One of the errors from bikeshed pertain to AudioNode.connect (but for when the destination is an AudioParam). Or perhaps we just have the argumentdef blocks all wrong for these methods.

rtoy commented 3 years ago

Teleconf: Leaving this here for now but there's probably not much we can do since it seems a bikeshed issue.