rewrite ir_Argo by using bit field

[siriuslzx]

I think it must meet 3 conditions when using bit field:

1. Don't straddle bytes unless necessary;
2. Don't exceed the width of the underlying type;
3. Follow the Object's alignment requirement
   see https://en.cppreference.com/w/cpp/language/bit_field#Notes
   https://en.cppreference.com/w/cpp/language/object#Alignment

for example:

    // Byte 2~4
    uint64_t          :3;
    uint64_t Mode     :3;
    uint64_t Temp     :5;  // straddle byte 2 and 3
    uint64_t Fan      :2;
    uint64_t RoomTemp :5;  // straddle byte 3 and 4
    uint64_t Flap     :3;  // SwingV
    uint64_t          :3;  // OnTimer, maybe hours

The field "Temp" straddle byte 2 and 3 and "RoomTemp" straddle byte 3 and 4, it means the bytes 2~4 should be taken as a whole, so you can't use uint8_t or uint16_t to hold them. If you use uint32_t, it's size is 4 and it's alignment must >=4, so you can't place it at byte 2. Finally, I use uint64_t start at byte 0.

[crankyoldgit]

LGTM, as usual. Review Comment is non-blocking, but I'd appreciate your opinion/thoughts.

[crankyoldgit]

@siriuslzx In future, please use the "Squash and Merge" option when merging PRs.

See: https://github.com/crankyoldgit/IRremoteESP8266/wiki/Library-Maintainers-Guide#merging-your-pull-request
(I only just added that after I saw you didn't use that.)

[siriuslzx]

@siriuslzx In future, please use the "Squash and Merge" option when merging PRs.

See: https://github.com/crankyoldgit/IRremoteESP8266/wiki/Library-Maintainers-Guide#merging-your-pull-request
(I only just added that after I saw you didn't use that.)

got it

[crankyoldgit]

FYI, the changes mentioned above have now been included in the new v2.7.11 release of the library.