stan-dev/stan

Clean up code: `.str().length()`

Aberta

#2.577 aberto em 10 de jul. de 2018

Ver no GitHubGuia do colaborador
 (13 comentários) (0 reação) (1 responsável)C++ (387 forks)batch import
code cleanupgood first issue

Métricas do repositório

Stars
 (2.761 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Summary:

There are many instances of code that looks like:

if (msg.str().length() > 0)
  logger.info(msg);

We should either:

  • have the info() method ignore empty messages in general
  • wrap the call in a function

Additional Information:

Originally from this PR comment: https://github.com/stan-dev/stan/pull/2570#discussion_r201151286

Current Version:

v2.17.1

Guia do colaborador

Direção de pesquisa
Identifique todas as ocorrências do padrão `if (msg.str().length() > 0) logger.info(msg);` no código. Avalie duas abordagens: (1) modificar o método info() para ignorar mensagens vazias, ou (2) encapsular o padrão em uma função auxiliar. Implemente a solução escolhida de forma consistente e atualize os testes conforme necessário.
Pilha de tecnologia
cpp
Domain
backend
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
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.
60