dotnet/runtime

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

Open

#26.096 aberto em 7 de mai. de 2018

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)C# (5.445 forks)batch import
area-System.Reflection.Metadatahelp wantedtest-enhancement

Métricas do repositório

Stars
 (17.886 stars)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue a especificação ECMA 335 para restrições na largura da coluna MethodList quando a contagem de métodos excede 0xFFFF. Revise a infraestrutura de teste existente para System.Reflection.Metadata no repositório dotnet/runtime, provavelmente em src/libraries/System.Reflection.Metadata/tests. Use MetadataBuilder para emitir um assembly PE com 0xFFFF métodos e um último tipo sem métodos. Verifique os comentários da issue para discussão adicional sobre null como um valor legal.
Pilha de tecnologia
csharp
Domain
backend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C#ECMA 335 spec
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45