@gbhug5a Thank you very much for testing sendSharp. 👍

I adjusted the code, such that 0x100 is default and for 0x200 you must call sendSharp2().

Before I release it, can you be so kind and test, what the non inverted/inverted sequence for a single and multiple repeats is.
And maybe also the length of the gaps, if they are not standard.

I'm not sure I understand your request. The three-time transfer (normal/inverted/normal) is what the Sharp remote does for even the briefest keypress. That's the minimum transmission. So are you asking what happens if the key is held down?

The Sharp remote break between the three phases is about 45000-46000. Yours is about 40700, but works fine. (I am using timer1 to send.)

Sorry for being unspecific.

So are you asking what happens if the key is held down?

Yes exactly, that is what i want to know 😀.

When the key is held down, the remote continues to send alternating inverted/normal packets with the same 45000 break between them. Here's the volume up key held down for a couple seconds:

On Off On Off On Off On Off

282, 1810, 284, 758, 284, 758, 284, 758,
284, 758, 284, 758, 284, 758, 284, 1808,
287, 755, 287, 1806, 288, 754, 288, 754,
288, 754, 288, 1805, 279, 762, 280, 47317,
278, 1814, 280, 762, 280, 762, 280, 762,
280, 762, 280, 1812, 282, 1811, 284, 758,
284, 1808, 286, 756, 286, 1806, 289, 1804,
279, 1813, 282, 760, 282, 1809, 285, 43109,
287, 1804, 280, 762, 280, 762, 280, 762,
280, 762, 280, 762, 280, 762, 280, 1812,
282, 760, 282, 1810, 284, 758, 284, 758,
284, 758, 284, 1808, 286, 756, 286, 47171,

341, 1751, 281, 761, 281, 761, 281, 761,
280, 762, 280, 1812, 283, 1809, 285, 757,
285, 1808, 286, 756, 286, 1806, 289, 1803,
281, 1811, 283, 759, 283, 1809, 285, 42970,
280, 1812, 282, 760, 282, 759, 283, 759,
282, 760, 282, 760, 282, 760, 282, 1810,
284, 758, 284, 1808, 287, 755, 287, 755,
287, 755, 287, 1805, 289, 752, 280, 47175,

283, 1809, 286, 756, 286, 755, 287, 755,
287, 755, 287, 1805, 289, 1803, 281, 761,
281, 1811, 283, 759, 283, 1810, 284, 1808,
286, 1806, 289, 753, 288, 1803, 281, 42972,
288, 1804, 279, 763, 279, 763, 279, 763,
279, 763, 279, 763, 279, 763, 279, 1813,
281, 761, 281, 1811, 283, 759, 283, 759,
283, 758, 284, 1808, 286, 755, 288, 47167,

281, 1810, 284, 758, 284, 758, 285, 757,
285, 757, 285, 1807, 287, 1805, 279, 763,
279, 1813, 281, 761, 281, 1811, 283, 1809,
285, 1807, 288, 754, 288, 1803, 281, 42972,
279, 1813, 281, 760, 282, 760, 282, 760,
282, 760, 282, 760, 282, 760, 282, 1810,
284, 758, 284, 1808, 287, 755, 287, 617839

Thanks!
I updated ir_Denon.hpp
Two questions left.

1. Is a command with only 1 normal/inverted accepted by the TV?
2. Is a command with 2 normal/inverted accepted as one press?
   This would simplify programming and backward compatibility 😀

The answer to your first question is yes, the TV does accept it. But I found it to be not as reliable. Sometimes I press a key and it doesn't take. That may be because there is no long start bit in Sharp. In any case, the normal/inverted/normal protocol seems to work best.

On the second question, I don't really have a way to test that. If I do any version of 1 repeat, the TV only recognizes the first keypress. But I suspect that's because it is coded not to do anything with a repeated key until it has been repeating for some time, like half a second. But I will try to test more with the volume up key to see if I can find anything definitive.

To follow the behavior of the remote, it looks like the code would be this:

sendPulseDistanceWidthData(&DenonProtocolConstants, tData, DENON_BITS);

while (tNumberOfCommands > 0) {

    // Inverted autorepeat frame
    delay(DENON_AUTO_REPEAT_DISTANCE / MICROS_IN_ONE_MILLI);
    sendPulseDistanceWidthData(&DenonProtocolConstants, tInvertedData, DENON_BITS);

    delay(DENON_AUTO_REPEAT_DISTANCE / MICROS_IN_ONE_MILLI);
    sendPulseDistanceWidthData(&DenonProtocolConstants, tData, DENON_BITS);


    tNumberOfCommands--;
    }
}

Thanks, but I am quite reluctant to change the code, since is must be backwards compatible and maybe other remotes work differently.
I added a comment "Maybe minimal number of repeats is 1 for a press to be reliable detected by some devices" to help users.

In your new version, shouldn't the last parameter in this be "2" instead of "true"?

void IRsend::sendSharp2(uint8_t aAddress, uint8_t aCommand, int_fast8_t aNumberOfRepeats) {
sendDenon(aAddress, aCommand, aNumberOfRepeats, true);
}

It seems to late for me...
Thank you for being so attentive 👍

I just got a Denon Remote and documented and changed the code to send 3 + n*2 repeat frames 😀