cginternals / cginternals/cppexpose

Variant array naming confusion

Ouverte
#48 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
decision required enhancement
Langage dominant
C
Étoiles
28
Forks
12
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

`Variant::isArray()` and `Variant::asArray()` mean different things by 'Array' (array by means of the typed-library vs. `VariantArray` aka `std::vector`). `isVariantArray` is the actual type query function corresponding to `asArray`, so `asArray` should probably be renamed `asVariantArray`. Same goes for `asMap` & `isVariantMap`.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

No files or tests are named. Start by locating Variant::isArray(), asArray(), isVariantArray, asMap(), and isVariantMap, then inspect their declarations and repository usages. Done means the naming consistently distinguishes typed-library arrays and VariantArray/VariantMap values without leaving mismatched call sites.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
tooling
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.