emscripten-core/emscripten

Name pthread workers for easier debugging

Open

#12 216 ouverte le 15 sept. 2020

Voir sur GitHub
 (12 commentaires) (3 réactions) (0 assignés)C++ (3 519 forks)batch import
good first bughelp wanted

Métriques du dépôt

Stars
 (27 361 stars)
Métriques de merge PR
 (Merge moyen 19j 10h) (147 PRs mergées en 30 j)

Description

This is a feature suggestion.

Workers accept an optional name which can be used for debugging, e.g. in DevTools we show the name in the Sources panel. Screen Shot 2020-09-15 at 10 55 05 am Screen Shot 2020-09-15 at 10 55 12 am

The name is also available to the worker via globalThis.name.

My suggestion is that in src/library_pthread.js, in allocateUnusedWorker when we call new Worker(), we give a useful name like pthread #1 etc. Right now threads just show up with the file name (which also means all the threads have the same name).

Guide contributeur

Direction de recherche
Localisez la fonction allocateUnusedWorker dans src/library pthread.js et ajoutez un paramètre name à l'appel new Worker() en utilisant l'index du worker.
Stack technique
javascript
Domaine
developer experience
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75