open-source-parsers/jsoncpp

Use std::ostringstream for numeric formatting

Open

#419 opened on 2016年2月11日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)C++ (7,542 stars) (2,602 forks)batch import
bughelp wanted

説明

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.

コントリビューターガイド

技術スタック
cpp
領域
backend
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
basic C++std::ostringstream familiarity
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65
調査方針
First, identify all uses of std::snprintf in the codebase, likely in files under src/lib json/. Replace each with std::ostringstream for numeric formatting. Ensure the output matches the expected JSON format and that existing tests pass. Refer to issue #418 for context on problems with snprintf. No open PRs or assignees, making this a clean task for a first time contributor.