Skip to content

Commit

Permalink
Give IRmacros.h smaller scope to avoid impacting projects using IRrem…
Browse files Browse the repository at this point in the history
…oteESP8266 (#1857)

Most projects use IRremoteESP8266 by including the IRremoteESP8266.h file.
To avoid unexpectedly impacting these projects it is safer to move the include to only where it is to be used.

Ref #1853
Ref #1851
  • Loading branch information
tonhuisman authored Aug 17, 2022
1 parent 48f2db2 commit bb78783
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/IRremoteESP8266.h
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@
#include <string>
#endif // UNIT_TEST

#include "IRmacros.h"

// Library Version Information
// Major version number (X.x.x)
#define _IRREMOTEESP8266_VERSION_MAJOR 2
Expand Down
2 changes: 2 additions & 0 deletions src/IRtext.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@
#include "IRremoteESP8266.h"
#include "i18n.h"

#include "IRmacros.h"

#ifndef PROGMEM
#define PROGMEM // Pretend we have the PROGMEM macro even if we really don't.
#endif
Expand Down

0 comments on commit bb78783

Please sign in to comment.