apple/swift-nio

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

Open

#2.506 aberto em 17 de ago. de 2023

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Swift (749 forks)batch import
good first issuekind/enhancement

Métricas do repositório

Stars
 (8.453 stars)
Métricas de merge de PR
 (Mesclagem média 8d 5h) (18 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
A implementação deve adicionar novos inicializadores e métodos a `NIOLoopBound` e `NIOLoopBoundBox` no módulo NIO. A issue sugere usar `assertInEventLoop` em vez de `preconditionInEventLoop` nas variantes não verificadas, e marcá las com `@inlinable` para otimização em modo de release. Os contribuidores devem revisar a implementação existente nos links de documentação fornecidos e considerar quaisquer implicações de segurança de thread.
Pilha de tecnologia
swift
Domain
backendperformance
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.
3
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Swift concurrencyNIOLoopBound APIassert vs precondition
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60