dotnet/runtime

Properly factor DllImports into Common\src\Interop

Ouverte

#28 035 ouverte le 30 nov. 2018

Voir sur GitHubGuide contributeur
 (23 commentaires) (0 réaction) (0 personne assignée)C# (5 445 forks)batch import
area-Metaenhancementgood first issuehelp wantedtracking

Métriques du dépôt

Stars
 (17 886 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Examinez les directives d'interopérabilité liées dans le problème, puis pour chaque bibliothèque non cochée, identifiez toutes les déclarations DllImport dans le code source, factorisez les dans des fichiers appropriés sous src/Common/src/Interop regroupés par plateforme et bibliothèque, et incluez ces fichiers sources dans les assemblys concernés.
Stack technique
csharp
Domaine
developer experience
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC#.NET interop
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45