Skip to content

Remove extra_info argument from js_optimizer.py. NFC#26981

Open
sbc100 wants to merge 1 commit into
emscripten-core:mainfrom
sbc100:remove_extra_info
Open

Remove extra_info argument from js_optimizer.py. NFC#26981
sbc100 wants to merge 1 commit into
emscripten-core:mainfrom
sbc100:remove_extra_info

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented May 20, 2026

This had no users. The only user of run_on_file is run_js_optimizer which doesn't pass this argument.

This had no users.  The only user of `run_on_file` is `run_js_optimizer`
which doesn't pass this argument.
@sbc100 sbc100 requested a review from kripken May 20, 2026 00:24
@sbc100
Copy link
Copy Markdown
Collaborator Author

sbc100 commented May 20, 2026

The js_optimizer tool seems like it only used for wasm2js output these days. Should we just remove it and rely on the existing closure tools etc, or is it serving some important function?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant