ArkScript-lang / ArkScript-lang/modules

Switch nlohmann/json for rapidjson

Đang mở
#59 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
good first issue help wanted priority/medium
Ngôn ngữ chính
C++
Star
7
Fork
4
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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.

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Bắt đầu bằng cách xác định module json và submodule nlohmann/json hiện tại của nó; xem xét cách các bài kiểm thử sử dụng API của module và cách các dependency được cài đặt thông qua CPM. Thay thế dependency mà không thay đổi API của module, xóa submodule và xác nhận các bài kiểm thử hiện có đều chạy qua; bổ sung bài kiểm thử khi cần.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
cpp
Lĩnh vực
backend
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
25/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.