dotnet/runtime

Consider renaming COM* types in clr native code to CLR*

Open

#86.145 aberto em 12 de mai. de 2023

Ver no GitHub
 (20 comments) (2 reactions) (0 assignees)C# (5.445 forks)batch import
area-VM-coreclrhelp wantedin-pr

Métricas do repositório

Stars
 (17.886 stars)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

Description

Currently, there are still some native code elements named with COM prefix from the COM+ era, like COMFloat and COMDouble:

https://github.com/dotnet/runtime/blob/a5d13e24cb374b55c906061f825d864efec7f3fb/src/coreclr/classlibnative/inc/floatdouble.h#L10

Can we rename them to things like CLRDouble, to make a clear distinction with COM interop? Are there any tools depending on these names?

Guia do colaborador

Direção de pesquisa
Examine o arquivo src/coreclr/classlibnative/inc/floatdouble.h e arquivos relacionados para identificar todos os tipos prefixados com COM. Procure por quaisquer dependências externas ou usos desses nomes no repositório e em todo o ecossistema .NET. Revise a discussão nos comentários da issue (20 comentários) para ver se foi alcançado um consenso ou se houve contribuição dos mantenedores. Determine se a renomeação quebraria alguma ferramenta ou API e, em caso afirmativo, planeje uma estratégia de migração.
Pilha de tecnologia
csharpcpp
Domain
backend
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65