Do not unconditionally profile (that was a miscommit). Instead, make a PROFILE=1...

[movit] / Makefile

diff --git a/Makefile b/Makefile
index 69e9ef44287ab4109acbc9d2e138f6508e9ea389..c71b8f3b304fb829d81f2eae8f904d35b8268f7c 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -6,8 +6,13 @@ CXXFLAGS=-Wall -g -I$(GTEST_DIR)/include $(shell pkg-config --cflags eigen3 )
 LDFLAGS=-lSDL -lSDL_image -lGL -lrt -lpthread
 RANLIB=ranlib
 
+ifeq ($(PROFILE),1)
+CXXFLAGS += -fprofile-arcs -ftest-coverage
+LDFLAGS += -fprofile-arcs -ftest-coverage
+endif
+
 DEMO_OBJS=demo.o
-TESTS=effect_chain_test mix_effect_test
+TESTS=effect_chain_test mix_effect_test gamma_expansion_effect_test
 
 # Core.
 LIB_OBJS=util.o widgets.o effect.o effect_chain.o
@@ -47,11 +52,13 @@ gtest_sdl_main.o: gtest_sdl_main.cpp
 
 # Unit tests.
 effect_chain_test: effect_chain_test.o $(TEST_OBJS) libmovit.a
-       $(CXX) -o $@ effect_chain_test.o $(TEST_OBJS) libmovit.a $(LDFLAGS)
+       $(CXX) -o $@ $^ $(LDFLAGS)
 mix_effect_test: mix_effect_test.o $(TEST_OBJS) libmovit.a
-       $(CXX) -o $@ mix_effect_test.o $(TEST_OBJS) libmovit.a $(LDFLAGS)
+       $(CXX) -o $@ $^ $(LDFLAGS)
+gamma_expansion_effect_test: gamma_expansion_effect_test.o $(TEST_OBJS) libmovit.a
+       $(CXX) -o $@ $^ $(LDFLAGS)
 
-OBJS=$(DEMO_OBJS) $(LIB_OBJS) $(GDEMO_OBJS)
+OBJS=$(DEMO_OBJS) $(LIB_OBJS) $(TEST_OBJS) $(TESTS:=.o)
 
 # A small demo program.
 demo: libmovit.a $(DEMO_OBJS)
@@ -72,8 +79,10 @@ clean:
        $(RM) demo $(TESTS) libmovit.a $(OBJS) $(DEPS)
 
 check: $(TESTS)
+       FAIL=0; \
        for TEST in $(TESTS); do \
-           ./$$TEST; \
-       done
+           ./$$TEST || FAIL=1; \
+       done; \
+       exit $$FAIL
 
 .PHONY: clean check all