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

Removing unused headers and functions #19

Open
ghenry opened this issue Feb 3, 2022 · 2 comments
Open

Removing unused headers and functions #19

ghenry opened this issue Feb 3, 2022 · 2 comments

Comments

@ghenry
Copy link
Collaborator

ghenry commented Feb 3, 2022

Some of these includes can be deleted, but I'm presuming the unused functions are for users of the library?

Screenshot from 2022-02-03 12-20-19

We could clear these warnings by using them in a test cases if possible.

@GrumpyOldTroll
Copy link
Owner

Yes, test cases that use these is I think the right action for those functions.

Was there a file for the unused includes? Pretty sure those are needed in some cases, but I believe it if it's saying they're present in some places they're not needed.

@ghenry
Copy link
Collaborator Author

ghenry commented Feb 18, 2022 via email

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

No branches or pull requests

2 participants