PHAREHUB/PHARE

macro ends in semi colon

Closed

#1.094 aberto em 28 de out. de 2025

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (33 forks)auto 404
good first issuerefactoring

Métricas do repositório

Stars
 (78 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

this allows for users to not have a semi colon at the call site of this macro, which is somewhat inconsistent

https://github.com/PHAREHUB/PHARE/blob/238185fa1027a7915ca44d13fabddd7534620289/src/core/logger.hpp#L18

Guia do colaborador

Direção de pesquisa
Remova o ponto e vírgula final da definição da macro em src/core/logger.hpp, linha 18, para permitir que os locais de chamada não tenham ponto e vírgula.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90