swiftlang/swift

[SR-9789] Use "pretty" nullability in generated ObjC headers

Aberta

#52.214 aberto em 29 de jan. de 2019

Ver no GitHubGuia do colaborador
 (5 comentários) (0 reação) (1 responsável)Swift (10.719 forks)batch import
PrintAsClangcompilergood first issueimprovement

Métricas do repositório

Stars
 (69.989 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description
Previous ID SR-9789
Radar None
Original Reporter @belkadan
Type Improvement
Votes 0
Component/s Compiler
Labels Improvement, PrintAsObjC, StarterBug
Assignee @adellibovi
Priority Medium

md5: 0bc925759fd7590c0a91c67c7a5de68f

Issue Description:

Objective-C supports two "pretty" forms for nullability qualifiers: within the parentheses of a method's parameters or return type:

- (id _Nullable)computate:(int * _Nonnull)values;

- (nullable id)computate:(nonnull int *)values;

And in the property attributes list for a property:

@property id _Nullable owner;

@property (nullable) id owner;

PrintAsObjC should adopt these where valid; right now it only uses them in hardcoded cases for instancetype.

Guia do colaborador

Direção de pesquisa
Observe a implementação do PrintAsObjC no compilador Swift, localize onde os qualificadores de nulidade são emitidos e atualize os para usar as formas 'bonitas' como `nullable` e `nonnull` dentro de parênteses ou atributos de propriedade. Comece examinando o uso hardcoded existente para `instancetype`.
Pilha de tecnologia
swiftobjective c
Domain
compilers
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
GitSwift
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80