apple/swift-nio

NIOLoopBound should get `unchecked` variants that are truly free in release mode

Offen

#2.506 geöffnet am 17.08.2023

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Swift (749 Forks)batch import
good first issuekind/enhancement

Repository-Metriken

Stars
 (8.453 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 9T 10h) (15 gemergte PRs in 30 T)

Beschreibung

NIOLoopBound(Box) is super useful when adopting Sendable but the eventLoop.preconditionInEventLoop() checks everywhere are expensive and -- for repeated accesses -- unnecessary.

There should be unchecked variants of init, get and set value that do eventLoop.assertInEventLoop() instead. Together with @inlinable that should become completely free in release mode (and still check in debug).

Docs:

Contributor Guide

Research-Richtung
Implementiere ungeprüfte Varianten von NIOLoopBound init, get und set, die eventLoop.assertInEventLoop() anstelle von preconditionInEventLoop() verwenden, und markiere sie als @inlinable. Stelle sicher, dass sie im Release Modus völlig kostenlos werden, während sie im Debug weiterhin prüfen.
Tech Stack
swift
Domain
backend
Issue Type
Funktion
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.
1-3 Stunden
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
Swift
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65