files-community/Files

Code Quality: Replace references to Vanara and WinForms with CsWin32

Fechada

#15.000 aberto em 19 de mar. de 2024

Ver no GitHubGuia do colaborador
 (11 comentários) (5 reações) (0 responsável)C# (2.719 forks)batch import
help wanted

Métricas do repositório

Stars
 (43.502 estrelas)
Métricas de merge de PR
 (Mesclagem média 11d 15h) (61 fundiu PRs em 30d)

Description

Description

We're working on utilizing Native Aot in the application, however, runtime marshaler is one of the blocker against this. Vanara uses loads of runtime marshaler via DllImport and ComImport and we're working on replacing it with CsWin32, ultimately removing its reference completely.

Concerned code

  • Vanara references
  • DllImport we explicitly use within the codebase

Gains

  • AoT compliant codebase
  • Less 3rd party references

Requirements

  • Replace DllImport with CsWin32

Comments

CsWin32 requires you to add interfaces, functions, constants, enums, snd etc in NativeMethods.txt placed in Files.App.CsWin32. #define'd constants also need to be written in that text file and use like "PInvoke.CMF_DEFAULT" (They don't get placed in a particular namespace). For more info, download ILSpy and win32metadata.winmd in order to see the full projections.

Guia do colaborador

Direção de pesquisa
Examine a base de código existente em busca de referências a Vanara e uso explícito de DllImport. Compreenda a estrutura de NativeMethods.txt em Files.App.CsWin32. Para cada DllImport, encontre a API Win32 correspondente nos metadados do Windows SDK e adicione a ao NativeMethods.txt. Use ILSpy e win32metadata.winmd para descobrir as assinaturas adequadas. Substitua as chamadas DllImport pelos wrappers CsWin32 gerados (por exemplo, PInvoke.SomeFunction). Certifique se de que o aplicativo ainda seja compilado e executado corretamente. Consulte os comentários da issue para orientações adicionais sobre constantes #define.
Pilha de tecnologia
csharp
Domain
desktop
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Knowledge of C#Understanding of P/InvokeFamiliarity with CsWin32
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30