open-source-parsers/jsoncpp

Use std::ostringstream for numeric formatting

Open

#419 aberto em 11 de fev. de 2016

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (2.602 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (7.542 stars)
Métricas de merge de PR
 (Mesclagem média 18h 26m) (10 fundiu PRs em 30d)

Description

We've had lots of problems with std::snprintf. E.g. #418.

As BillyGoto says:

Could we switch to std::ostringstream instead? Its numeric formatting is actually specified in terms of printf's output, and it's everywhere.

Guia do colaborador

Direção de pesquisa
Primeiro, identifique todos os usos de std::snprintf na base de código, provavelmente em arquivos sob src/lib json/. Substitua cada um por std::ostringstream para formatação numérica. Certifique se de que a saída corresponda ao formato JSON esperado e que os testes existentes passem. Consulte a issue #418 para contexto sobre problemas com snprintf. Não há PRs abertos ou responsáveis atribuídos, tornando esta uma tarefa limpa para um contribuidor iniciante.
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-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
C++STL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75