emscripten-core/emscripten

Use console.log to print values of args of emscripten_set_canvas_element_size

Ouverte

#14 551 ouverte le 29 juin 2021

Voir sur GitHubGuide contributeur
 (3 commentaires) (1 réaction) (0 personne assignée)C++ (3 519 forks)batch import
good first bughelp wanted

Métriques du dépôt

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

Description

the type of log message is not considered an error. It is small but makes us confusing as see the console. We might think that an error returned as calling emscripten_set_canvas_element_size. Should be changed to log or info.

https://github.com/emscripten-core/emscripten/blob/9dd9c35fdddc19b9a843cd9406657fc37e16dc45/src/library_html5.js#L2494

Guide contributeur

Direction de recherche
Localisez le fichier src/library html5.js à la ligne 2494. Changez l'appel de console.error à console.log ou console.info pour les arguments de emscripten set canvas element size. Il s'agit d'un changement d'une seule ligne. Assurez vous de la cohérence avec les autres journaux similaires dans la même fonction ou fichier.
Stack technique
javascript
Domaine
frontenddeveloper experience
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.
1
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.
85