Skip to content

[docs] Update descriptions for undefined symbol settings. NFC - #27598

Open
sbc100 wants to merge 1 commit into
emscripten-core:mainfrom
sbc100:docs-undefined-symbols
Open

[docs] Update descriptions for undefined symbol settings. NFC#27598
sbc100 wants to merge 1 commit into
emscripten-core:mainfrom
sbc100:docs-undefined-symbols

Conversation

@sbc100

@sbc100 sbc100 commented Aug 22, 2026

Copy link
Copy Markdown
Collaborator

Update and clarify the doc comments in src/settings.js for ERROR_ON_UNDEFINED_SYMBOLS and WARN_ON_UNDEFINED_SYMBOLS, and regenerate site/source/docs/tools_reference/settings_reference.rst.

@sbc100
sbc100 requested a review from kripken August 22, 2026 23:47
Update and clarify the doc comments in `src/settings.js` for
`ERROR_ON_UNDEFINED_SYMBOLS` and `WARN_ON_UNDEFINED_SYMBOLS`, and
regenerate `site/source/docs/tools_reference/settings_reference.rst`.
@sbc100
sbc100 force-pushed the docs-undefined-symbols branch from dbec33f to 8ad67d2 Compare August 22, 2026 23:48
to 0, in which case if an undefined function is called a runtime error will
occur. Any undefined symbols that are listed in EXPORTED_FUNCTIONS will also
be reported.
Disable this setting to allow undefined functions at link time. Instead of

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Disable this setting to allow undefined functions at link time. Instead of
Disable this setting to allow undefined functions at link time. Instead

@kripken kripken left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm otherwise

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.

2 participants