dotnet/runtime

Share AssemblyName.EscapeString with UriHelper.EscapeString

Open

#44.752 aperta il 16 nov 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-System.Reflectionhelp wanted

Metriche repository

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

Descrizione

It seems at some point the AssemblyName version was copied from UriHelper and hasn't kept up with any changes, including its rewrite in https://github.com/dotnet/runtime/commit/94acc996414dcf6b4b60847775e0247fd959eaa8. We should get rid of the old unsafe implementation entirely and just share the code if possible. cc: @stephentoub

Guida contributor

Direzione di ricerca
Confronta le implementazioni di AssemblyName.EscapeString e UriHelper.EscapeString nel repository dotnet/runtime. Identifica il vecchio codice unsafe e le ottimizzazioni recenti. Proponi un refactoring per condividere la logica comune.
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#.NET SDK
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70