Improve type safety of internal classes
まだ誰も着手していません。
評価
調査の方向性
まず、この issue の影響を受ける内部クラスと基底クラスから派生クラスへのキャストを洗い出し、次にそれらのオブジェクトがインストルメンテーションエンジンに入る箇所を追跡します。現在の継承パターンとインターフェースパターンを、提案されている仮想継承および派生クラスでの QI アプローチと比較します。関連する安全でないキャストに対処し、影響を受けるパスが互換性を維持していれば完了です。
索引モデルが issue の本文から書いたものです。
説明
There are a number of places where we cast from a base class to a derived class in our internal code, which could cause problems if a caller passes us an instance of an interface that we did not create internally. I suggest we:
-
Update our code to use virtual inheritance (instead of multiple inheritance). This will help us catch locations where we cast from a base class.
-
QI for the derived class. This will give us type safety.
- 主要言語
- C++
- スター
- 104
- フォーク
- 53
- 平均マージ
- 3時間 34分
- マージ済み PR（30日）
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/CLRInstrumentationEngine のほかの issue
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
microsoft/CLRInstrumentationEngine#169 · コメント 2 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
microsoft/CLRInstrumentationEngine#526 · リアクション 1 件 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
microsoft/CLRInstrumentationEngine の issue をすべて見る
似ている issue
-
Status: Awaiting triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
espressif/arduino-esp32#12984 ·
メンテナーはふだん 1 日以内に返信
-
torch_ops/logprob.cu does not compile with the serving container's nvcc (13.3.73); check_torch_ops.py cannot run as shipped対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1時間未満 初心者へのやさしさ 72/100
ashhart/TensorFold#535 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
agent:Windows bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信