dotnet/runtime

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

Open

#26.096 aperta il 7 mag 2018

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-System.Reflection.Metadatahelp wantedtest-enhancement

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga la specifica ECMA 335 per i vincoli sulla larghezza della colonna MethodList quando il numero di metodi supera 0xFFFF. Esamina l'infrastruttura di test esistente per System.Reflection.Metadata nel repository dotnet/runtime, probabilmente sotto src/libraries/System.Reflection.Metadata/tests. Usa MetadataBuilder per emettere un assembly PE con 0xFFFF metodi e un ultimo tipo senza metodi. Controlla i commenti dell'issue per ulteriori discussioni su null come valore legale.
Tech stack
csharp
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C#ECMA 335 spec
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45