cxplat: use of inline functions and macros
メンテナーはふだん 4 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- リファクタリング
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- cpp
調査の方向性
まず issue #89 の議論と、現在別々のファイルに定義されている cxplat 関数を確認します。inline または macro 対応の候補となる単純な関数を特定し、何を移動するか決める前にフットプリントとメモリへの影響を検討します。この作業は、検討結果が文書化され、レビュー済みになった時点で完了とします。
索引モデルが issue の本文から書いたものです。
説明
In #89, comments ehre:
Nick: General question though: Can/should we move stuff to be header only (via macros or possibly inline functions) instead of functions defined in separate files? Ideally the linker should do a good job of eliminating overhead, but it doesn't always do it perfect.
Dave: The downside of macros/inlines is that if they're non-trivial and used a lot, then they can increase the footprint and memory requirements. So I would say: yes for trivial functions, no for complex functions. Can do a pass in a separate PR.
- 主要言語
- C++
- スター
- 112
- フォーク
- 36
- 平均マージ
- 3日 6時間
- マージ済み PR（30日）
- 3
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/usersim のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
メンテナーはふだん 4 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
microsoft/usersim#275 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 4 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
メンテナーはふだん 4 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
メンテナーはふだん 4 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
メンテナーはふだん 4 日以内に返信
microsoft/usersim の issue をすべて見る
似ている issue
-
bug intrinsic medium priority rt_lib
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
OpenPrinting/splix#33 · コメント 3 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
sumatrapdfreader/sumatrapdf#6417 · コメント 3 件 ·
-
[BUG] Cython cache evicts most recent instead of oldest entry on cleanup対応中かも @scoder が今日担当しました。 オープンBuild System defect
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100