preamble.js: emcc-generated functions have length 0

Métriques du dépôt

Stars

27 361 stars (27 361 stars)

Métriques de merge PR

Merge moyen 19j 10h (Merge moyen 19j 10h)147 PRs mergées en 30 j (147 PRs mergées en 30 j)

Description

echo >test.c 'int foo (int x) { return 0; }
' &&
echo >test.js 'assert (
_foo .length == 01,
[ '\''foo has'\'',
_foo .length,
'\''arguments'\''
] .join ('\'' '\''
))' &&
emcc test.c --post-js test.js -sEXPORTED_FUNCTIONS=_foo &&
node a.out

Assertion failed: foo has 0 arguments

Guide contributeur

Direction de recherche

Étudiez preamble.js dans le dépôt Emscripten, en particulier la logique d'encapsulation des fonctions qui exporte les fonctions C. Le bogue est que les fonctions exportées ont .length 0, ce qui signifie que la fonction JavaScript générée manque de paramètres formels. Examinez comment les arguments sont passés et si la signature de la fonction doit inclure des déclarations de paramètres pour correspondre aux paramètres de la fonction C. Regardez les mécanismes 'dynCall' ou similaires qui gèrent les arguments variadiques. La correction implique probablement de modifier le wrapper pour déclarer le nombre correct de paramètres en fonction de la signature originale de la fonction.

Stack technique

cjavascript

Domaine

tooling

Type d'issue

Bug

DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.

3

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-3 heures

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Active

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

CJavaScript

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

30