chakra-core/ChakraCore

JSRT: JsDeleteIndexedProperty does not tell successful or not

Open

#504 aperta il 10 mar 2016

Vedi su GitHub
 (10 commenti) (0 reazioni) (0 assegnatari)JavaScript (1374 fork)batch import
APIsBugSeverity: 3good first issuehelp wanted

Metriche repository

Star
 (9000 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Confronta l'implementazione di JsDeleteIndexedProperty e JsDeleteProperty nel codice sorgente di ChakraCore. Modifica JsDeleteIndexedProperty per restituire un codice JsErrorCode che indica il successo/fallimento e aggiungi un parametro useStrictRules. Assicurati che la modifica sia coerente con i pattern API esistenti e aggiorna tutti i chiamanti interni.
Tech stack
ccpp
Dominio
apibackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60