ArkScript-lang / ArkScript-lang/modules

Switch nlohmann/json for rapidjson

オープン
#59 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
good first issue help wanted priority/medium
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。