emscripten-core/emscripten

Use console.log to print values of args of emscripten_set_canvas_element_size

Offen

#14.551 geöffnet am 29.06.2021

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (3.519 Forks)batch import
good first bughelp wanted

Repository-Metriken

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

Beschreibung

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

Contributor Guide

Research-Richtung
Suchen Sie die Datei src/library html5.js in Zeile 2494. Ändern Sie den Aufruf von console.error zu console.log oder console.info für die Argumente von emscripten set canvas element size. Dies ist eine einzeilige Änderung. Stellen Sie die Konsistenz mit anderen ähnlichen Log Ausgaben in derselben Funktion oder Datei sicher.
Tech Stack
javascript
Domain
frontenddeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
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.
85