nodejs/llnode

Improved message for inspection of empty objects

Aberta

#253 aberto em 13 de dez. de 2018

Ver no GitHubGuia do colaborador
 (0 comentário) (1 reação) (0 responsável)C++ (101 forks)github user discovery
enhancementgood first issue

Métricas do repositório

Stars
 (1.167 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Encontre o código que imprime a representação de objetos do heap V8 e adicione uma verificação para objetos vazios para exibir uma mensagem mais clara.
Pilha de tecnologia
cpp
Domain
developer experiencetooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70