dotnet/runtime

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

Open

#26.096 geöffnet am 7. Mai 2018

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-System.Reflection.Metadatahelp wantedtest-enhancement

Repository-Metriken

Stars
 (17.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die ECMA 335 Spezifikation auf Einschränkungen bezüglich der Spaltenbreite von MethodList, wenn die Methodenanzahl 0xFFFF überschreitet. Überprüfen Sie die vorhandene Testinfrastruktur für System.Reflection.Metadata im dotnet/runtime Repository, wahrscheinlich unter src/libraries/System.Reflection.Metadata/tests. Verwenden Sie MetadataBuilder, um eine PE Assembly mit 0xFFFF Methoden und einem letzten Typ ohne Methoden zu erzeugen. Prüfen Sie die Issue Kommentare auf zusätzliche Diskussionen zu null als legalem Wert.
Tech Stack
csharp
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C#ECMA 335 spec
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45