Skip to content

Commit 72c21fe

Browse files
Make chars unsigned by default, matching ESP8266 (#6013)
Force GCC to run with -funsigned-char during host tests to make the PC match the default behaviour used by the xtensa GCC port. As noted in #6010. Thanks @MichaelBrunn3r
1 parent c5efb92 commit 72c21fe

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Diff for: tests/host/Makefile

+2-2
Original file line numberDiff line numberDiff line change
@@ -145,8 +145,8 @@ FLAGS += -DLWIP_IPV6=0
145145
FLAGS += -DHOST_MOCK=1
146146
FLAGS += -DNONOSDK221=1
147147
FLAGS += $(MKFLAGS)
148-
CXXFLAGS += -std=c++11 -fno-rtti $(FLAGS)
149-
CFLAGS += -std=c99 $(FLAGS)
148+
CXXFLAGS += -std=c++11 -fno-rtti $(FLAGS) -funsigned-char
149+
CFLAGS += -std=c99 $(FLAGS) -funsigned-char
150150
LDFLAGS += -coverage $(OPTZ) -g $(M32)
151151
VALGRINDFLAGS += --leak-check=full --track-origins=yes --error-limit=no --show-leak-kinds=all --error-exitcode=999
152152
CXXFLAGS += -Wno-error=format-security # cores/esp8266/Print.cpp:42:24: error: format not a string literal and no format arguments [-Werror=format-security] -- (os_printf_plus(not_the_best_way))

0 commit comments

Comments
 (0)