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

The implementation and declaration of some functions differs from the function call #2101

Closed
MarkKoshelev opened this issue Jun 4, 2024 · 4 comments · Fixed by #2102
Closed
Assignees
Labels
bug Pending Confirmation Waiting for confirmation from user

Comments

@MarkKoshelev
Copy link

I encountered this when my AC Haier was not working correctly.
src/IRac.cpp
1460: void IRac::haier176(IRHaierAC176 *ac, const haier_ac176_remote_model_t model,
const bool on, const stdAc::opmode_t mode,
const bool celsius, const float degrees,
const stdAc::fanspeed_t fan,
const stdAc::swingv_t swingv,
const stdAc::swingh_t swingh,
const bool turbo, const bool quiet, const bool filter,
const int16_t sleep)

3289: haier176(&ac, (haier_ac176_remote_model_t)send.model, send.power,
send.mode, send.celsius, send.degrees, send.fanspeed,
send.swingv, send.swingh, send.turbo, send.filter, send.sleep);

The call of function haier176 dont have "const bool quiet" argument

crankyoldgit added a commit that referenced this issue Jun 5, 2024
@crankyoldgit crankyoldgit self-assigned this Jun 5, 2024
crankyoldgit added a commit that referenced this issue Jun 5, 2024
@crankyoldgit crankyoldgit added the Pending Confirmation Waiting for confirmation from user label Jun 6, 2024
@crankyoldgit
Copy link
Owner

@MarkKoshelev I think I've fixed it. Can you please confirm it now operates as expected in the master branch?

@MarkKoshelev
Copy link
Author

@MarkKoshelev I think I've fixed it. Can you please confirm it now operates as expected in the master branch?

You have fixed ony one function (haier176). But the same bug is in a nubber of functions.
I see them in haierYrwo2 also.
Please check all functions declaration and implementation. All haierXXX function have this bug.

@NiKiZe
Copy link
Collaborator

NiKiZe commented Jul 3, 2024

@MarkKoshelev I think I've fixed it. Can you please confirm it now operates as expected in the master branch?

You have fixed ony one function (haier176). But the same bug is in a nubber of functions. I see them in haierYrwo2 also. Please check all functions declaration and implementation. All haierXXX function have this bug.

Feel free to create one or more PRs 😉

crankyoldgit added a commit that referenced this issue Jul 3, 2024
Quiet mode operation was missing in the function calls of `haierYrwo2()` and `haier160()` thus passing the wrong arguments to the function.
Add the `quiet` parameter in fixes this.

Fixes #2101
@MarkKoshelev
Copy link
Author

MarkKoshelev commented Jul 3, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Pending Confirmation Waiting for confirmation from user
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants