chakra-core/ChakraCore

JSRT: JsDeleteIndexedProperty does not tell successful or not

Open

#504 geöffnet am 10. März 2016

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.374 Forks)batch import
APIsBugSeverity: 3good first issuehelp wanted

Repository-Metriken

Stars
 (9.000 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Vergleichen Sie die Implementierung von JsDeleteIndexedProperty und JsDeleteProperty im ChakraCore Quellcode. Ändern Sie JsDeleteIndexedProperty, um einen JsErrorCode zurückzugeben, der Erfolg/Misserfolg anzeigt, und fügen Sie einen useStrictRules Parameter hinzu. Stellen Sie sicher, dass die Änderung mit vorhandenen API Mustern konsistent ist und aktualisieren Sie alle internen Aufrufer.
Tech Stack
ccpp
Domain
apibackend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60