emscripten-core/emscripten

Use console.log to print values of args of emscripten_set_canvas_element_size

Open

#14.551 aperta il 29 giu 2021

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)C++ (3519 fork)batch import
good first bughelp wanted

Metriche repository

Star
 (27.361 star)
Metriche merge PR
 (Merge medio 19g 10h) (147 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Individua il file src/library html5.js alla riga 2494. Cambia la chiamata da console.error a console.log o console.info per gli argomenti di emscripten set canvas element size. Questa è una modifica di una sola riga. Assicurati la coerenza con altri log simili nella stessa funzione o file.
Tech stack
javascript
Dominio
frontenddeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85