apple/swift-nio

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

Aberta

#2.506 aberto em 17 de ago. de 2023

Ver no GitHubGuia do colaborador
 (5 comentários) (0 reação) (0 responsável)Swift (749 forks)batch import
good first issuekind/enhancement

Métricas do repositório

Stars
 (8.453 estrelas)
Métricas de merge de PR
 (Mesclagem média 9d 10h) (15 fundiu PRs em 30d)

Description

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:

Guia do colaborador

Direção de pesquisa
Implemente variantes unchecked de init, get e set do NIOLoopBound que usem eventLoop.assertInEventLoop() em vez de preconditionInEventLoop(), e marque as como @inlinable. Garanta que se tornem completamente gratuitas no modo release, enquanto ainda verificam no debug.
Pilha de tecnologia
swift
Domain
backend
Tipo Issue
Funcionalidade
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.
1-3 horas
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
Swift
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65