nodejs/llnode

Improved message for inspection of empty objects

Aperta

#253 aperta il 13 dic 2018

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)C++ (101 fork)github user discovery
enhancementgood first issue

Metriche repository

Star
 (1167 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Right now, when you try to inspect an empty object, this is what you see:

(llnode) v8 i 0x000038e4d66492e1
0x000038e4d66492e1:<Object: Object >

This is a bit confusing, upon further inspection it looks like 0x000038e4d66492e1 is an empty object, {}. A more user friendly message could be very helpful.

PS. Big thanks to all the contributors on this project. Has been a lifesaver on multiple occasions.

Guida contributor

Direzione di ricerca
Trova il codice che stampa la rappresentazione degli oggetti heap V8 e aggiungi un controllo per oggetti vuoti per visualizzare un messaggio più chiaro.
Tech stack
cpp
Dominio
developer experiencetooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70