Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle circular dependencies #8

Open
KamranAsif opened this issue Oct 9, 2017 · 1 comment
Open

Handle circular dependencies #8

KamranAsif opened this issue Oct 9, 2017 · 1 comment
Assignees
Milestone

Comments

@KamranAsif
Copy link
Owner

http://w3c.github.io/html/infrastructure.html#section-structuredserializeinternal

@KamranAsif KamranAsif self-assigned this Oct 9, 2017
@KamranAsif KamranAsif added this to the V1 release milestone Oct 13, 2017
@KamranAsif
Copy link
Owner Author

const foo = {};
foo.name = foo;
const bar = {};
bar.name = bar;

Trivial cases

const source = {prop: foo};
const revision = {prop: foo};

Objects are same reference, so we don't walk down tree.

const source = {prop: foo};
const revision = {prop: [non object/array]};

Both props need to be objects, otherwise we return revision value.

Complex cases

const source = {prop: foo};
const revision = {prop: bar};

Not sure what we should return right now..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant