emscripten-core/emscripten

Use console.log to print values of args of emscripten_set_canvas_element_size

Open

#14,551 opened on 2021年6月29日

GitHub で見る
 (3 comments) (1 reaction) (0 assignees)C++ (3,519 forks)batch import
good first bughelp wanted

Repository metrics

Stars
 (27,361 stars)
PR merge metrics
 (平均マージ 19d 10h) (30d で 147 merged PRs)

説明

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

コントリビューターガイド

調査方針
ファイル src/library html5.js の 2494 行目を特定します。emscripten set canvas element size の引数に対する呼び出しを console.error から console.log または console.info に変更します。これは単一行の変更です。同じ関数またはファイル内の他の類似したロギングとの一貫性を確保してください。
技術スタック
javascript
領域
frontenddeveloper experience
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
1
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
JavaScript
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85