dotnet/runtime

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

Open

#86.145 geöffnet am 12. Mai 2023

Auf GitHub ansehen
 (20 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-VM-coreclrhelp wantedin-pr

Repository-Metriken

Stars
 (17.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Untersuchen Sie die Datei src/coreclr/classlibnative/inc/floatdouble.h und verwandte Dateien, um alle COM präfixierten Typen zu identifizieren. Suchen Sie nach externen Abhängigkeiten oder Verwendungen dieser Namen im Repository und im gesamten .NET Ökosystem. Überprüfen Sie die Diskussion in den Issue Kommentaren (20 Kommentare), um festzustellen, ob ein Konsens erzielt wurde oder ob Input von Maintainern vorliegt. Bestimmen Sie, ob eine Umbenennung Tools oder APIs brechen würde, und planen Sie gegebenenfalls eine Migrationsstrategie.
Tech Stack
csharpcpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65