dotnet/roslyn

CLR bug with static variable in struct could be warned by the compiler

Open

#10.126 aperta il 26 mar 2016

Vedi su GitHub
 (27 commenti) (2 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-CompilersFeature - Warning WavesFeature Requesthelp wanted

Metriche repository

Star
 (20.414 star)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

The following code compiles, but throws a TypeLoadException when executed.

using System;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            var x = MyStruct.Empty;
        }

        public struct MyStruct
        {
            public static readonly MyStruct? Empty = null;
        }
    }
}

Since Empty is static, this is not a cyclic struct layout issue. It would be nice if the compiler produced warnings that tells you that you are likely to run into this CLR bug at runtime

[Note from @gafter this is due to CLR bug https://github.com/dotnet/coreclr/issues/4049. See also https://github.com/dotnet/coreclr/issues/7957 and https://github.com/dotnet/coreclr/issues/22553 for other situations where the warning would be helpful]

Guida contributor

Direzione di ricerca
Investiga come aggiungere un avviso per i campi statici di tipi struct nullable nel compilatore, facendo riferimento ai bug CLR (coreclr #4049, #7957, #22553). L'avviso dovrebbe attivarsi quando una struct contiene un campo statico del proprio tipo nullable.
Tech stack
csharp
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60