ArkScript-lang / ArkScript-lang/modules
Switch nlohmann/json for rapidjson
- 主要言語
- C++
- スター
- 7
- フォーク
- 4
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
We would like to switch to rapidjson (MIT) for the json module, instead of nlohmann/json.
Boost would be faster, but an MIT license is better (benchmarks: https://230.jsondocs.prtest.cppalliance.org/libs/json/doc/html/json/benchmarks.html).
The module API shouldn't change, however more functions could be added, depending on the needs. Tests should pass, more can be added (because we never have enough tests).
The json submodule will have to be removed, rapidjson could be installed through CPM.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
まず json モジュールと現在の nlohmann/json サブモジュールの場所を確認し、テストがその API をどのように使用しているか、また CPM を通じて依存関係がどのようにインストールされているかを確認します。モジュール API を変更せずに依存関係を置き換え、サブモジュールを削除し、既存のテストがパスすることを確認します。必要に応じてテストを追加します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- backend
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 25/100