emscripten-core/emscripten

Cryptic `sig is undefined` error in reportUndefinedSymbols() if function signature not available

Open

#17.283 geöffnet am 21. Juni 2022

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (27.361 Stars) (3.519 Forks)batch import
good first bughelp wanted

Beschreibung

Discussed in https://github.com/emscripten-core/emscripten/discussions/17266

Originally posted by johnwason June 18, 2022 Currently if a function signature is not defined, reportUndefinedSymbols() will simple raise an error sig is undefined, which is not at all helpful. An error handler should be added to check if the value passed to addFunction() does not have sig defined, and report something more useful to the user. See #17264

Contributor Guide

Tech Stack
ccppjavascriptwasm
Domain
backendtoolingdeveloper experience
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScriptEmscripten basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
Locate the `reportUndefinedSymbols()` function in Emscripten's JavaScript runtime code. The function currently raises a cryptic 'sig is undefined' error when a function signature is not provided. Review the related discussion (#17266) and issue (#17264) for background. Add a check to verify that `sig` is defined before using it, and provide a more informative error message indicating which function lacks a signature. The fix is likely in a single file within the Emscripten source tree.