swiftlang/swift

Diagnostic for `static nonmutating` should be improved

Open

#77.835 aberto em 25 de nov. de 2024

Ver no GitHub
 (11 comments) (0 reactions) (1 assignee)Swift (10.719 forks)batch import
bugcompilerdeclarationsdiagnostics qualitygood first issuemutatingstatic declarationsswift 6.1type checker

Métricas do repositório

Stars
 (69.989 stars)
Métricas de merge de PR
 (Mesclagem média 7d 6h) (556 fundiu PRs em 30d)

Description

Description

No response

Reproduction

struct S {
  nonmutating static func foo() {}
}

430 | 
431 | struct S {
432 |   nonmutating static func foo() {}
    |   `- error: static functions must not be declared mutating [static_functions_not_mutating]
433 | }
434 |

Expected behavior

An example approach to improving this error message is to list both modifiers.

We can also tailor it to call out the modifier in question as it currently does in the case of mutating. The drawback of this option is that it conveys less information about language grammar.s

Environment

Swift version 6.1-dev (26e888e45e6ede4)

Additional information

No response

Guia do colaborador

Direção de pesquisa
Localize o diagnóstico para 'static functions not mutating' no código fonte do compilador Swift, provavelmente em lib/Sema ou em um arquivo de diagnóstico similar. Modifique a mensagem de erro para listar ambos os modificadores 'static' e 'nonmutating' conforme sugerido na issue. Revise a discussão para obter contexto adicional sobre o comportamento esperado.
Pilha de tecnologia
swift
Domain
developer experience
Tipo Issue
Refatoração
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.
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 programming language
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65