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