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

Fix warning introduced in 74e68af #845

Merged

Conversation

pguyot
Copy link
Collaborator

@pguyot pguyot commented Oct 5, 2023

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.

Nitpicking: can you tell which warning message are you fixing in the commit message?
Also, I didn't check the commit subject, but please make sure to keep it short by using just 74e68af as commit hash.

Remove unused static prototype declaration as the function was eventually moved
to posix_nifs.c.

Signed-off-by: Paul Guyot <[email protected]>
@pguyot pguyot force-pushed the w40/fix-warning-on-unused-static-declaration branch from 4b40b32 to 2fe639f Compare October 6, 2023 05:08
@pguyot pguyot changed the title Fix warning introduced in 74e68af1791d5ed72d6a6063bd90870a7620dac4 Fix warning introduced in 74e68af Oct 6, 2023
@fadushin fadushin merged commit 9e4c220 into atomvm:master Oct 6, 2023
81 checks passed
@pguyot pguyot deleted the w40/fix-warning-on-unused-static-declaration branch October 6, 2023 17:53
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.

3 participants