nodejs/llnode

Improved message for inspection of empty objects

Ouverte

#253 ouverte le 13 déc. 2018

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)C++ (101 forks)github user discovery
enhancementgood first issue

Métriques du dépôt

Stars
 (1 167 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Trouvez le code qui affiche la representation des objets du tas V8 et ajoutez une verification pour les objets vides afin d'afficher un message plus clair.
Stack technique
cpp
Domaine
developer experiencetooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
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.
70