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

[movit] / Makefile

diff --git a/Makefile b/Makefile
index aaa9896ec36294e3c4a22d96a1190813e084f576..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
@@ -50,6 +55,8 @@ effect_chain_test: effect_chain_test.o $(TEST_OBJS) libmovit.a
        $(CXX) -o $@ $^ $(LDFLAGS)
 mix_effect_test: mix_effect_test.o $(TEST_OBJS) libmovit.a
        $(CXX) -o $@ $^ $(LDFLAGS)
+gamma_expansion_effect_test: gamma_expansion_effect_test.o $(TEST_OBJS) libmovit.a
+       $(CXX) -o $@ $^ $(LDFLAGS)
 
 OBJS=$(DEMO_OBJS) $(LIB_OBJS) $(TEST_OBJS) $(TESTS:=.o)
 
@@ -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