emscripten-core/emscripten

Use console.log to print values of args of emscripten_set_canvas_element_size

Open

#14.551 aberto em 29 de jun. de 2021

Ver no GitHub
 (3 comments) (1 reaction) (0 assignees)C++ (3.519 forks)batch import
good first bughelp wanted

Métricas do repositório

Stars
 (27.361 stars)
Métricas de merge de PR
 (Mesclagem média 19d 10h) (147 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Localize o arquivo src/library html5.js na linha 2494. Altere a chamada de console.error para console.log ou console.info para os argumentos de emscripten set canvas element size. Esta é uma alteração de uma linha. Garanta consistência com outros logs semelhantes na mesma função ou arquivo.
Pilha de tecnologia
javascript
Domain
frontenddeveloper experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85