Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

docs: fix smalloc.dispose() example #8625

Closed
wants to merge 1 commit into from
Closed

docs: fix smalloc.dispose() example #8625

wants to merge 1 commit into from

Conversation

vkurchatkin
Copy link

Also couple of additions about dispose and limitations of smalloc'ed objects.

/cc @trevnorris

@@ -48,6 +48,9 @@ possible options are listed in `smalloc.Types`. Example usage:

// { '0': 0, '1': 0.1, '2': 0.2 }

It is not possible to freeze, seal and prevent extensions of objects with external data using `Object.freeze`,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrapped to 80 columns?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

Also couple of additions about `dispose`
and limitations of smalloc'ed objects.

After `dispose()` is called object still behaves as one with external data, for
example `smalloc.hasExternalData()` returns `true`.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does make me sad V8 doesn't provide a way to remove this.

trevnorris pushed a commit that referenced this pull request Oct 27, 2014
Also couple of additions about dispose and limitations of smalloc'ed
objects.

Fixes: #8625
Reviewed-by: Trevor Norris <trev.norris@gmail.com>
@trevnorris
Copy link

Thanks much. Merged in 15aa47e.

@trevnorris trevnorris closed this Oct 27, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants