Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Fedora warning fix #38

Merged
merged 3 commits into from
May 30, 2023
Merged

Fedora warning fix #38

merged 3 commits into from
May 30, 2023

Conversation

bkmgit
Copy link
Contributor

@bkmgit bkmgit commented May 28, 2023

#37

bkmgit added 2 commits May 28, 2023 14:47
@bkmgit
Compare unsigned long types
6571181 
Comparing signed int and unsigned long gives compiler warning
@bkmgit
Remove unused variables
355d481
@svenstaro
Copy link
Owner

svenstaro commented May 30, 2023 •
edited
Loading

Thanks though I believe had_arguments is actually required. You didn't remove the other assignment of had_arguments.

@bkmgit
Remove unused variable
e3190e7 
Additional had_arguments occurrence causing failed build
@bkmgit
Copy link
Contributor Author

bkmgit commented May 30, 2023

Thanks. Removed.

@svenstaro svenstaro merged commit 5121437 into svenstaro:master May 30, 2023
@bkmgit bkmgit deleted the fedora-warning-fix branch May 30, 2023 08:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bkmgit @svenstaro