glideapps/quicktype

C++: use different integer type

Open

#2 180 ouverte le 16 févr. 2023

Voir sur GitHub
 (2 commentaires) (2 réactions) (0 assignés)TypeScript (10 867 stars) (968 forks)batch import
C++customizationgood first issue

Description

When using a JSON Schema to generate C++ code, the generated class always uses int64_t for integer properties. In my other code this frequently leads to compiler warnings when I want to assign an integer property value to a C++ int, because the compiler warns about casts from int64_t to int (because we build with -Wc++11-narrowing and -Wshorten-64-to-32).

Is there a way to specify the C++ data type that should be used for a property? Then I could explicitly change the generated type to int (or short or char) where necessary.

Guide contributeur

Stack technique
cpptypescript
Domaine
tooling
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
C++JSON Schemaquicktype basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
Investigate the C++ code generation module in quicktype (e.g., `src/quicktype core/language/Cpp.ts`). Look for existing mechanisms for type customization in other languages (e.g., Python's `type mapping`). Consider adding a new option to allow the user to specify the integer type via schema annotations or a command line flag. Review the discussion in the issue comments for any additional context.