dotnet/runtime

Remove unsafe code from number parsing

Open

#10.397 aperta il 29 mag 2018

Vedi su GitHub
 (11 commenti) (3 reazioni) (0 assegnatari)C# (5445 fork)batch import
Cost:Marea-System.Numericshelp wantedreduce-unsafe

Metriche repository

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

Descrizione

There is heavy use of unsafe code in number.parsing.cs and calling code (eg at https://github.com/dotnet/runtime/blob/110282c71b3f7e1f91ea339953f4a0eba362a62c/src/libraries/System.Private.CoreLib/src/System/Number.Parsing.cs#L255). This could be rewritten with ReadOnlySpan<char> to eliminate the char * resulting in safer code that is also easier to read.

Relates to https://github.com/dotnet/coreclr/pull/17808

Guida contributor

Direzione di ricerca
Riscrivi il codice non sicuro in Number.Parsing.cs utilizzando ReadOnlySpan<char> invece di char*. Segui il pattern dalla PR referenziata (dotnet/coreclr#17808) e assicurati di aggiornare tutti i chiamanti.
Tech stack
csharp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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.
70