dotnet/runtime

Add test coverage for ECMA-335 corner around non-optional Lists

オープン

#26,096 opened on 2018/05/07

GitHub で見るコントリビューターガイド
 (4 件のコメント) (0 件のリアクション) (0 人の担当者)C# (5,445 件のフォーク)batch import
area-System.Reflection.Metadatahelp wantedtest-enhancement

Repository metrics

Stars
 (17,886 個のスター)
PR merge metrics
 (平均マージ 12d 11h) (30d で 661 merged PRs)

説明

Use System.Reflection.Metadata to emit the following:

  • A module with exactly 0xFFFF methods
  • The last type in the module has no methods

This will hit a corner case in the ECMA-335 spec where the TypeDef record for the last type should end up with a MethodList value of 0x10000, but per the spec, the MethodList column is only 2 bytes wide (because the method count fits in 2 bytes), making the value fall out of range.

This also affects other similar columns, such as TypeDef.FieldList, and MethodDef.ParamList.

Per the spec, null might be a legal value to put there in that case.

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

調査方針
メソッド数が 0xFFFF を超える場合の MethodList 列の幅に関する制約について、ECMA 335 仕様を調査します。dotnet/runtime リポジトリ内の System.Reflection.Metadata の既存のテストインフラストラクチャ（おそらく src/libraries/System.Reflection.Metadata/tests 以下）を確認します。MetadataBuilder を使用して、0xFFFF 個のメソッドを含み、最後の型にメソッドがない PE アセンブリを出力します。null が有効な値であることに関する追加の議論については、issue のコメントを確認してください。
技術スタック
csharp
領域
backend
Issue 種別
テスト
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
C#ECMA 335 spec
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
45