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

rpt_config.c: Add missing votermode load parameter #523

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

mkmer
Copy link
Collaborator

@mkmer mkmer commented Feb 28, 2025

Found another missing initialization. The clue was that we were initializing votertype 2x.
Also, ASL2 inits these with default 0

@mkmer mkmer added the bug Something isn't working label Feb 28, 2025
@mkmer mkmer self-assigned this Feb 28, 2025
@Allan-N
Copy link
Collaborator

Allan-N commented Feb 28, 2025

Also, ASL2 inits these with default 0

I believe that all of the variables start off with zero'd out values.

@mkmer mkmer force-pushed the missing-votermode-parameter branch from d5b0cb5 to 2eb2812 Compare February 28, 2025 20:17
@mkmer mkmer force-pushed the missing-votermode-parameter branch from 2eb2812 to 2483a9e Compare February 28, 2025 20:47
@Allan-N Allan-N merged commit f995658 into master Feb 28, 2025
3 checks passed
@mkmer mkmer deleted the missing-votermode-parameter branch February 28, 2025 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants