dotnet/runtime

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

Open

#26,096 建立於 2018年5月7日

在 GitHub 查看
 (4 留言) (0 反應) (0 負責人)C# (5,445 fork)batch import
area-System.Reflection.Metadatahelp wantedtest-enhancement

倉庫指標

Star
 (17,886 star)
PR 合併指標
 (平均合併 12天 11小時) (30 天內合併 661 個 PR)

描述

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.

貢獻者指南

研究方向
研究 ECMA 335 規範，瞭解當方法數量超過 0xFFFF 時對 MethodList 欄位寬度的限制。檢視 dotnet/runtime 儲存庫中 System.Reflection.Metadata 的現有測試基礎設施，可能在 src/libraries/System.Reflection.Metadata/tests 目錄下。使用 MetadataBuilder 產生一個包含 0xFFFF 個方法且最後一個類型沒有方法的 PE 組件。檢視問題評論中關於 null 作為合法值的額外討論。
技術棧
csharp
領域
backend
議題類型
測試
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
半天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
大致清晰
前置要求
C#ECMA 335 spec
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
45