Use async/await in libwebaudio.js. NFC

[sbc100]

No description provided.

[sbc100]

View with "hide whitespace"

[juj]

What does using 'async' provide here? It increases code size, but otherwise seems nonfunctional?

[sbc100]

Just modernizing the code, but maybe not worth in this case? If I can't find a way to make it is at least size neutral maybe I will just abandon.

[sbc100]

(I do think it makes the code more readable in the modern form, avoid extra indentation etc, but not worth it in this case probably)

[cwoffenden]

My understanding is, async doesn't gain us anything here because these functions are never passed to promises, and only used from wasm. I'm sure it's the correct, idiomatic way of writing JS though.

[sbc100]

My understanding is, async doesn't gain us anything here because these functions are never passed to promises, and only used from wasm. I'm sure it's the correct, idiomatic way of writing JS though.

Correct this would just be an NFC, stylistic change.

[cwoffenden]

LGTM. The gzipped JS and wasm size stay the same.

[juj]

I would recommend avoiding this, as it is not necessary, does not provide benefit, and has a code size regression.