chakra-core/ChakraCore

JSRT: JsDeleteIndexedProperty does not tell successful or not

Open

#504 ouverte le 10 mars 2016

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)JavaScript (1 374 forks)batch import
APIsBugSeverity: 3good first issuehelp wanted

Métriques du dépôt

Stars
 (9 000 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

JsDeleteIndexedProperty API does not tell if the deletion is successful or not, unlike JsDeleteProperty which returns that info in "result". The former also misses "useStrictRules" flag which exists in the later.

JsDeleteIndexedProperty(JsValueRef object, JsValueRef index)

JsDeleteProperty(JsValueRef object, JsPropertyIdRef propertyId,
                 bool useStrictRules,
                 JsValueRef *result)

These 2 APIs should be comparable and consistent.

Guide contributeur

Direction de recherche
Comparez l'implémentation de JsDeleteIndexedProperty et JsDeleteProperty dans le code source de ChakraCore. Modifiez JsDeleteIndexedProperty pour qu'elle retourne un code JsErrorCode indiquant le succès/échec et ajoutez un paramètre useStrictRules. Assurez vous que la modification est cohérente avec les modèles d'API existants et mettez à jour tous les appelants internes.
Stack technique
ccpp
Domaine
apibackend
Type d'issue
Bug
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-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60