Skip to content

Conversation

@siriuslzx
Copy link
Collaborator

  1. remove const variables like k__size and k__offset and use bit field instead;
  2. add "const" to get__() api;
  3. remove "this->"
  4. add getFan() and setFan()

PS:My PRs are decorative rather than constructive, so I think there is no need to add my name to the copyright section. And I'm happy to be on the contributors list.

Copy link
Owner

@crankyoldgit crankyoldgit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it. Keep it up.

@crankyoldgit
Copy link
Owner

PS:My PRs are decorative rather than constructive, so I think there is no need to add my name to the copyright section. And I'm happy to be on the contributors list.

I agree, however the previous PR had you do some substantial refactoring, not just "window dressing". :-)

@siriuslzx siriuslzx merged commit e92a750 into crankyoldgit:master Feb 15, 2021
crankyoldgit added a commit that referenced this pull request Mar 24, 2021
_v2.7.16 (20210324)_

**[Features]**
- ToshibaAC: Swing handling and `setRaw()` improvements. (#1423 #1424 #1425)
- Support for XMP (Xfinity) protocol. (#1414 #1422)
- ToshibaAC: Adjust inter-message gap timing to improve matching. (#1420 #1421)
- Ecoclim: Add detailed A/C support (#1397 #1415)

**[Misc]**
- [ESP32] Fix `addApbChangeCallback(): duplicate func` kernel msgs (#1434 #1435)
- refactor ir_Fujitsu (#1419)
- refactor ir_Whirlpool (#1416)
- refactor ir_Vestel (#1413)
- refactor ir_Trotec (#1412)
@crankyoldgit crankyoldgit mentioned this pull request Mar 24, 2021
crankyoldgit added a commit that referenced this pull request Mar 24, 2021
## _v2.7.16 (20210324)_

**[Features]**
- ToshibaAC: Swing handling and `setRaw()` improvements. (#1423 #1424 #1425)
- Support for XMP (Xfinity) protocol. (#1414 #1422)
- ToshibaAC: Adjust inter-message gap timing to improve matching. (#1420 #1421)
- Ecoclim: Add detailed A/C support (#1397 #1415)

**[Misc]**
- [ESP32] Fix `addApbChangeCallback(): duplicate func` kernel msgs (#1434 #1435)
- refactor ir_Fujitsu (#1419)
- refactor ir_Whirlpool (#1416)
- refactor ir_Vestel (#1413)
- refactor ir_Trotec (#1412)
@crankyoldgit
Copy link
Owner

FYI, the aforementioned changes have been included in the new v2.7.16 release for the library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants