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++ (3.519 Forks)batch import
good first bughelp wanted

Repository-Metriken

Stars
 (27.361 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 19T 10h) (147 gemergte PRs in 30 T)

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

Research-Richtung
Füge in reportUndefinedSymbols() eine Prüfung hinzu, um sicherzustellen, dass die Eigenschaft 'sig' vorhanden ist, bevor darauf zugegriffen wird, und gib eine aussagekräftige Fehlermeldung aus, die den Benutzer dazu anleitet, die Funktionssignatur zu definieren.
Tech Stack
javascript
Domain
backend
Issue Type
Fehler
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70