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

Document and add specifications for several nifs and bifs #706

Conversation

pguyot
Copy link
Collaborator

@pguyot pguyot commented Jul 28, 2023

Document and add specifications of many nifs and bifs used in standard libraries.

See #701

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

@pguyot pguyot marked this pull request as draft July 29, 2023 06:23
Copy link
Collaborator Author

@pguyot pguyot left a comment

Choose a reason for hiding this comment

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

Not so sure we need to add it as a nif.

@pguyot pguyot force-pushed the w30/implement-erlang-get-and-document-put-erase branch from af875b2 to 1c881e0 Compare August 5, 2023 21:59
@pguyot pguyot changed the title Implement erlang:get/1 and document put/2 and erase/1 Document and add specifications for several nifs and bifs Aug 5, 2023
@pguyot pguyot marked this pull request as ready for review August 5, 2023 22:01
See atomvm#701
Also add tests for erlang:get/1, erlang:put/2 and erlang:erase/1

Signed-off-by: Paul Guyot <[email protected]>
@pguyot pguyot force-pushed the w30/implement-erlang-get-and-document-put-erase branch from 1c881e0 to a3d5131 Compare August 5, 2023 22:07
@fadushin fadushin merged commit a1f9bea into atomvm:master Aug 6, 2023
72 checks passed
@pguyot pguyot deleted the w30/implement-erlang-get-and-document-put-erase branch August 6, 2023 04:28
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

Successfully merging this pull request may close these issues.

2 participants