Clean up compiler warnings + Build pzstd on travis

.travis.yml

@@ -19,7 +19,7 @@ matrix:
           packages:
             - gcc-4.8
             - g++-4.8
-      env: PLATFORM="Ubuntu 12.04 container" CMD="make -C tests test-zstd_nolegacy && make clean && make zlibwrapper && make clean && make cmaketest && make -C contrib/pzstd googletest && make -C contrib/pzstd test && make -C contrib/pzstd clean"
+      env: PLATFORM="Ubuntu 12.04 container" CMD="make -C tests test-zstd_nolegacy && make clean && make zlibwrapper && make clean && make cmaketest && make clean && make -C contrib/pzstd pzstd && make -C contrib/pzstd googletest && make -C contrib/pzstd test && make -C contrib/pzstd clean"
     - os: linux
       sudo: false
       env: PLATFORM="Ubuntu 12.04 container" CMD="make usan"

contrib/pzstd/ErrorHolder.h

@@ -9,6 +9,7 @@
 #pragma once
 
 #include <atomic>
+#include <cassert>
 #include <stdexcept>
 #include <string>
 
@@ -47,9 +48,7 @@ class ErrorHolder {
   }
 
   ~ErrorHolder() {
-    if (hasError()) {
-      throw std::logic_error(message_);
-    }
+    assert(!hasError());
   }
 };
 }

contrib/pzstd/Makefile

@@ -10,7 +10,7 @@
 ZSTDDIR = ../../lib
 PROGDIR = ../../programs
 
-CPPFLAGS = -I$(ZSTDDIR) -I$(ZSTDDIR)/common -I$(ZSTDDIR)/dictBuilder -I$(PROGDIR) -I.
+CPPFLAGS = -I$(ZSTDDIR) -I$(ZSTDDIR)/common -I$(PROGDIR) -I.
 CXXFLAGS  ?= -O3
 CXXFLAGS  += -std=c++11
 CXXFLAGS  += $(MOREFLAGS)
@@ -55,7 +55,7 @@ main.o: main.cpp *.h utils/*.h
 	$(CXX) $(FLAGS) -c main.cpp -o $@
 
 pzstd: Pzstd.o SkippableFrame.o Options.o main.o libzstd.a 
-	$(CXX) $(FLAGS) $^ -o $@$(EXT)
+	$(CXX) $(FLAGS) $^ -o $@$(EXT) -lpthread
 
 googletest:
 	@git clone https://github.com/google/googletest

contrib/pzstd/SkippableFrame.cpp

@@ -7,7 +7,7 @@
  * of patent rights can be found in the PATENTS file in the same directory.
  */
 #include "SkippableFrame.h"
-#include "common/mem.h"
+#include "mem.h"
 #include "utils/Range.h"
 
 #include <cstdio>