open-source-parsers/jsoncpp

Use std::ostringstream for numeric formatting

Open

#419 ouverte le 11 févr. 2016

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C++ (2 602 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (7 542 stars)
Métriques de merge PR
 (Merge moyen 18h 26m) (10 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
Tout d'abord, identifiez toutes les utilisations de std::snprintf dans la base de code, probablement dans les fichiers sous src/lib json/. Remplacez chacune par std::ostringstream pour le formatage numérique. Assurez vous que la sortie correspond au format JSON attendu et que les tests existants passent. Reportez vous à l'issue #418 pour le contexte sur les problèmes avec snprintf. Il n'y a pas de PR ouverts ni de responsables attribués, ce qui en fait une tâche propre pour un contributeur débutant.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++STL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75