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 function declarations #681

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Schievel1
Copy link

Clang16 does not allow implitic function declarations by default. Right now I am going througt gentoos packages to make them built on clang16 in the future.

This overhauls the whole source code to make this build for clang16. Please take a look at where and how I declared functions and adapt to your liking. (I could not always find out what intentions of the structure of your code imply)

See also:
https://bugs.gentoo.org/875500
https://bugs.gentoo.org/870412

Signed-off-by: Pascal Jäger pascal.jaeger@leimstift.de

Signed-off-by: Pascal Jäger <pascal.jaeger@leimstift.de>
@Schievel1 Schievel1 marked this pull request as ready for review November 14, 2022 12:31
@Schievel1 Schievel1 marked this pull request as draft November 14, 2022 12:31
@lgtm-com
Copy link

lgtm-com bot commented Nov 14, 2022

This pull request fixes 29 alerts when merging 3218d31 into a798cba - view on LGTM.com

fixed alerts:

  • 29 for Implicit function declaration

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@Schievel1 Schievel1 marked this pull request as ready for review November 15, 2022 10:37
@sergiomb2
Copy link

also in Fedora rawhide with gcc-c++-13.2.1-5.fc40.x86_64 we need this patch , please rebase it

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