Skip to content

util: handle circular maps and sets in inspect() - #14776

Closed
aqrln wants to merge 1 commit into
nodejs:masterfrom
aqrln:deja-vu-is-a-glitch-in-the-matrix
Closed

util: handle circular maps and sets in inspect()#14776
aqrln wants to merge 1 commit into
nodejs:masterfrom
aqrln:deja-vu-is-a-glitch-in-the-matrix

Conversation

@aqrln

Copy link
Copy Markdown
Contributor

Handle maps and sets with circular references in util.inspect() the
way objects and arrays are treated in this case.

Fixes: #14758

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

util

Handle maps and sets with circular references in `util.inspect()` the
way objects and arrays are treated in this case.
Fixes: nodejs#14758
@aqrlnaqrln added the util Issues and PRs related to the built-in util module. label Aug 11, 2017
@aqrln

Copy link
Copy Markdown
ContributorAuthor

@BridgeAR oops, didn't see your PR, sorry. Just opened the issue and noticed it. Should I close mine?

@BridgeAR

Copy link
Copy Markdown
Member

@aqrln I do not mind either way. Your call

@aqrln

Copy link
Copy Markdown
ContributorAuthor

Closed in favor of #14775 since that PR was opened first.

@aqrlnaqrln closed this Aug 11, 2017
@aqrln
aqrln deleted the deja-vu-is-a-glitch-in-the-matrix branch August 12, 2017 16:54
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

utilIssues and PRs related to the built-in util module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

util.inspect not checking for loops in {,Weak}Map data

2 participants

@aqrln@BridgeAR