Skip to content

Commit 5d73ec0

Browse files
committed
Remove unused argument from autoResumeAudioContext
This fixes a TODO from 2022.
1 parent a45c791 commit 5d73ec0

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

src/lib/libcore.js

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1711,14 +1711,8 @@ addToLibrary({
17111711
// input events on, and registers a context resume() for them. This lets
17121712
// audio work properly in an automatic way, as browsers won't let audio run
17131713
// without user interaction.
1714-
// If @elements is not provided, we default to the document and canvas
1715-
// elements, which handle common use cases.
1716-
// TODO(sbc): Remove seemingly unused elements argument
1717-
$autoResumeAudioContext__docs: '/** @param {Object=} elements */',
1718-
$autoResumeAudioContext: (ctx, elements) => {
1719-
if (!elements) {
1720-
elements = [document, document.getElementById('canvas')];
1721-
}
1714+
$autoResumeAudioContext: (ctx) => {
1715+
var elements = [document, document.getElementById('canvas')];
17221716
['keydown', 'mousedown', 'touchstart'].forEach((event) => {
17231717
elements.forEach((element) => {
17241718
element?.addEventListener(event, () => {

test/codesize/test_codesize_hello_dylink_all.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
2-
"a.out.js": 245508,
2+
"a.out.js": 245504,
33
"a.out.nodebug.wasm": 574067,
4-
"total": 819575,
4+
"total": 819571,
55
"sent": [
66
"IMG_Init",
77
"IMG_Load",

0 commit comments

Comments
 (0)