open-source-parsers/jsoncpp

Use std::ostringstream for numeric formatting

Open

#419 aperta il 11 feb 2016

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (2602 fork)batch import
bughelp wanted

Metriche repository

Star
 (7542 star)
Metriche merge PR
 (Merge medio 18h 26m) (10 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Innanzitutto, identifica tutti gli utilizzi di std::snprintf nella codebase, probabilmente nei file sotto src/lib json/. Sostituisci ciascuno con std::ostringstream per la formattazione numerica. Assicurati che l'output corrisponda al formato JSON previsto e che i test esistenti passino. Fai riferimento all'issue #418 per il contesto sui problemi con snprintf. Non ci sono PR aperti o assegnatari, rendendo questo un compito pulito per un contributore alle prime armi.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++STL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75