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

Add support for list_to_integer/2 #1310

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

jakub-gonet
Copy link
Contributor

@jakub-gonet jakub-gonet commented Oct 11, 2024

Tested on unix port of AtomVM, some interactive testing and ./test-erlang test_list_to_integer.

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

Copy link
Collaborator

@bettio bettio left a comment

Choose a reason for hiding this comment

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

Everything looks good, except 2 formatting issues. I would also like to ask you to add this to our CHANGELOG.

src/libAtomVM/nifs.c Outdated Show resolved Hide resolved
src/libAtomVM/nifs.c Outdated Show resolved Hide resolved
@bettio
Copy link
Collaborator

bettio commented Oct 11, 2024

An additional request: please fixup existing commit rather than making an additional one, and if you can rebase on latest release-0.6 branch.

@jakub-gonet jakub-gonet changed the base branch from main to release-0.6 October 11, 2024 14:04
@jakub-gonet
Copy link
Contributor Author

Sorry for formatting, my editor wasn't fully setup for AVM. How releases work? You do development on latest release branch and copy commit over to main periodically?

@bettio
Copy link
Collaborator

bettio commented Oct 11, 2024

Sorry for formatting, my editor wasn't fully setup for AVM. How releases work? You do development on latest release branch and copy commit over to main periodically?

We do safe additions and fixes on release-0.6 branch, while new breaking changes (and any risky change) in main. I merge release-0.6 into main once a week.

@bettio bettio merged commit 85a3901 into atomvm:release-0.6 Oct 11, 2024
95 of 96 checks passed
@jakub-gonet jakub-gonet deleted the jgonet/list-to-binary-2 branch October 12, 2024 01:19
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