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

Remove unused variable. #74

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Remove unused variable. #74

merged 1 commit into from
Nov 10, 2023

Conversation

garthk
Copy link
Contributor

@garthk garthk commented Nov 7, 2023

nchan was set and maintained but never used in the parse_zones and parse_scanlist functions for six radios, resulting in this error with clang 15.0.0 and zig 0.11.0:

dm1801.c:1808:15: error: variable 'nchan' set but not used [-Werror,-Wunused-but-set-variable]
        int   nchan = 0;

`nchan` was set and maintained but never used in the `parse_zones` and
`parse_scanlist` functions for six radios, resulting in this error with
clang 15.0.0 and zig 0.11.0:

```plain
dm1801.c:1808:15: error: variable 'nchan' set but not used [-Werror,-Wunused-but-set-variable]
        int   nchan = 0;
```
@garthk
Copy link
Contributor Author

garthk commented Nov 7, 2023

As a side-effect, proof of compilation without other errors might help close out #65 and #66 which @keisisqrl fixed in #67?

@silseva silseva merged commit b03f0c8 into OpenRTX:master Nov 10, 2023
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