ArkScript-lang / ArkScript-lang/modules
Switch nlohmann/json for rapidjson
- Lingua principale
- C++
- Stelle
- 7
- Fork
- 4
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia individuando il modulo json e il suo attuale sottomodulo nlohmann/json; esamina come i test utilizzano la sua API e come vengono installate le dipendenze tramite CPM. Sostituisci la dipendenza senza modificare l'API del modulo, rimuovi il sottomodulo e verifica che i test esistenti passino; aggiungi test dove necessario.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- cpp
- Ambito
- backend
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 25/100