diff options
author | Jon Medhurst <tixy@linaro.org> | 2012-05-14 11:28:08 +0100 |
---|---|---|
committer | Jon Medhurst <tixy@linaro.org> | 2012-05-21 12:51:32 +0100 |
commit | 324bba217d7e9d31aa5247f6963efbf36ed602f9 (patch) | |
tree | ab57ae96219cb0f07ae6d643f0bb37a679491ba2 /tools/gator | |
parent | 414f4a7dfb33723bebe44c1af79ddd395ff2412d (diff) |
gator-daemon: Fix sequence-point compile error
In mxml/mxml-file.c in the expression:
(getc(fp) << 8) | getc(fp)
the C standard allows right-hand getc() to be called before left hand
one, which would cause a bug.
The compiler gives an warning about this and we are treating warnings as
errors this breaks the build with:
error: operation on 'fp->_r' may be undefined [-Werror=sequence-point]
We can fix this by rewriting the expressions in two parts, however the
code in question has licensing clauses for modified versions which have
not yet been resolved.
Therefore, for now, we will disable the problematic error in the
makefiles.
Signed-off-by: Jon Medhurst <tixy@linaro.org>
Diffstat (limited to 'tools/gator')
-rwxr-xr-x | tools/gator/daemon/Android.mk | 2 | ||||
-rw-r--r-- | tools/gator/daemon/Makefile | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/tools/gator/daemon/Android.mk b/tools/gator/daemon/Android.mk index 3ab55b91ad5..715e5b62ba1 100755 --- a/tools/gator/daemon/Android.mk +++ b/tools/gator/daemon/Android.mk @@ -3,7 +3,7 @@ include $(CLEAR_VARS) $(shell cd $(LOCAL_PATH);cat events_header.xml events-*\.xml events_footer.xml > events.xml;xxd -i events.xml > events_xml.h;xxd -i configuration.xml > configuration_xml.h)
-LOCAL_CFLAGS += -Wall -O3 -ftree-vectorize
+LOCAL_CFLAGS += -Wall -O3 -ftree-vectorize -Wno-error=sequence-point
LOCAL_SRC_FILES:= \
CapturedXML.cpp \
diff --git a/tools/gator/daemon/Makefile b/tools/gator/daemon/Makefile index 950e87060ac..c517a873728 100644 --- a/tools/gator/daemon/Makefile +++ b/tools/gator/daemon/Makefile @@ -18,7 +18,7 @@ GCC=$(CROSS_COMPILE)gcc # -std=c++98 is the 1998 c++ standard # -march=armv5t is required to set the minimum architecture # -mthumb-interwork is required for interworking to ARM or Thumb stdlibc -CFLAGS=-O3 -Wall -Werror +CFLAGS=-O3 -Wall -Werror -Wno-error=sequence-point TARGET=gatord C_SRC = $(wildcard mxml/*.c) CPP_SRC = $(wildcard *.cpp) |