Possible documentation bug (README.md, IrSender.begin)?

[Openuser87]

README.md states:

For sending, call IrSender.begin(ENABLE_LED_FEEDBACK); or IrSender.begin(DISABLE_LED_FEEDBACK); in setup().

Further it's stated that you must use the 3-argument syntax if IR_SEND_PIN is not defined.

But I could only get the 3-argument version to run, though I did define IR_SEND_PIN. When compiling with a single argument (e.g. IrSender.begin(ENABLE_LED_FEEDBACK);), the sender is doing nothing. I solved it just by chance (coming across this, although the "2-argument" syntax there didn't work with me either, but made me think in the right direction).

Maybe I am misunderstanding something, or this needs correction? Might save folks some time in the latter case...

[ArminJo]

Have defined IR_SEND_PIN before the line #include <IRremote.hpp>?
Have you read this?

[Openuser87]

No and no. 🙃

Thank you, just tested it - when defined before #include, it works! 👍 Sorry, not a programmer 😉

So, everything perfect then. Thanks for the quick reply and for the excellent library!