dotnet/runtime

Properly factor DllImports into Common\src\Interop

Aperta

#28.035 aperta il 30 nov 2018

Vedi su GitHubGuida contributor
 (23 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-Metaenhancementgood first issuehelp wantedtracking

Metriche repository

Star
 (17.886 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

We now have a bunch of libraries with their own DllImports, whereas the expected approach is that all DllImports in product src should be included from src\Common\src\Interop, appropriately factored by platform and library, and then those source files included into the relevant assemblies that need them.

  • Microsoft.Diagnostics.Tracing.EventSource.Redist
  • System.Data.SqlClient
  • System.Diagnostics.EventLog
  • System.DirectoryServices
  • System.DirectoryServices.AccountManagement
  • System.DirectoryServices.Protocols
  • System.Drawing.Common
  • System.IO.Compression
  • System.IO.Compression.Brotli
  • System.IO.Ports
  • System.Management
  • System.Reflection.Metadata
  • System.Runtime.Caching
  • System.Security.Cryptography.Csp
  • System.Security.Cryptography.Pkcs
  • System.Security.Cryptography.X509Certificates

This all needs to be refactored and dedup'd.

https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/interop-guidelines.md

Guida contributor

Direzione di ricerca
Rivedi le linee guida di interoperabilità collegate nell'issue, quindi per ogni libreria non selezionata, identifica tutte le dichiarazioni DllImport nel codice sorgente, suddividile in file appropriati sotto src/Common/src/Interop raggruppati per piattaforma e libreria, e includi quei file sorgente negli assembly pertinenti.
Tech stack
csharp
Dominio
developer experience
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
GitC#.NET interop
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45