Fix redefinition of "bool" type in library header

[legerch]

When including "Safestring.h" library for an ARDUINO_ARCH_SAM, boolean type is redefined to an integer. This will pollute our sketch since this redefinition is inside the header.
Fix it by using a custom boolean type name for this library

[drmpf]

In SafeString.h try replacing

#if defined(ARDUINO_ARCH_SAM)
#define bool int
#endif

#ifdef __cplusplus

#include <stdlib.h>
#include <string.h>
#include <ctype.h>

with

#ifdef __cplusplus

#include <stdbool.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

That change seems to work for SAM boards

[legerch]

In SafeString.h try replacing

#if defined(ARDUINO_ARCH_SAM)
#define bool int
#endif

#ifdef __cplusplus

#include <stdlib.h>
#include <string.h>
#include <ctype.h>

with

#ifdef __cplusplus

#include <stdbool.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

That change seems to work for SAM boards

I can confirm that this properly work, bool is a proper boolean type with those changes.
Do you want me to update this PR with this method instead ?

[drmpf]

No thats fine, I will do a new release with this fix and update the PlatformIO zips as well. Thanks for finding this.

[legerch]

Noted, thank you for maintaining this library :)

[drmpf]

fixed in V4.1.33