dotnet/runtime

Audit usages of Marshal.AllocHGlobal for ones that can be replaced with NativeMemory.Alloc

Offen

#54.297 geöffnet am 16.06.2021

Auf GitHub ansehenContributor Guide
 (6 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-Interop-coreclrhelp wanted

Repository-Metriken

Stars
 (17.886 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

#54006 introduced new NativeMemory APIs that wrap malloc, calloc, realloc, and aligned_alloc. We should audit existing usages of Marshal.AllocHGlobal for cases where it can be safely replaced with malloc.

Notably cases like StringToPtrHGlobal do not qualify as they are documented to be a call to LocalAlloc on Windows. However, on Unix and in cases where this is not a definitive contract, it should be safe to replace these calls with faster ones to NativeMemory.

Contributor Guide

Research-Richtung
Untersuchen Sie jede Verwendung von Marshal.AllocHGlobal im Code, identifizieren Sie diejenigen, die nicht unter Windows oder nicht als LocalAlloc dokumentiert sind, und ersetzen Sie sie bei Sicherheit durch NativeMemory.Alloc, überprüfen Sie dann Leistung und Korrektheit.
Tech Stack
csharp
Domain
performance
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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.
Meist klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60