jfpoilpret / fast-arduino-lib

C++ platform to build Arduino and AVR projects. Rationale is size and speed optimization.
GNU Lesser General Public License v2.1
60 stars 9 forks source link

Remove included streams.h to avoid conflicts with empty_streams.h #97

Closed jfpoilpret closed 2 years ago

jfpoilpret commented 2 years ago

In FastArduino library, several header files explicitly #include "streams.h" just to define specific operator<<() on types defined in these files. This has a spurious impact on programs that:

Since FastArduino allows end developers to use streams.h or empty_streams.h in their programs, their choice shall not prevent using some FastArduino features.

Thus any non-mandatory #include "streams.h" should be removed, and specific operator<<() shall be defined elsewhere or differently to fit either streams kind.

Concerned files are:

There are other headers including streams.h but they definitely need actual streams, so no change is needed for them.

jfpoilpret commented 2 years ago

Fix since PR #105