From c2511564cb4809b41699b88882fc5b3e5931070d Mon Sep 17 00:00:00 2001
From: Pere Mato <pere.mato@cern.ch>
Date: Tue, 4 Oct 2016 09:28:04 +0200
Subject: [PATCH] Fix (silence) warnings detected by clang 3.9

---
 interpreter/CMakeLists.txt              | 1 +
 math/matrix/src/TVectorT.cxx            | 2 +-
 roofit/roofitcore/src/BidirMMapPipe.cxx | 5 ++---
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/interpreter/CMakeLists.txt b/interpreter/CMakeLists.txt
index 409ffb5b015..49fe40cb11b 100644
--- a/interpreter/CMakeLists.txt
+++ b/interpreter/CMakeLists.txt
@@ -54,6 +54,7 @@ ROOT_ADD_C_FLAG(INTERP_ADDITIONAL_C_FLAGS -Wno-unused-parameter)
 ROOT_ADD_CXX_FLAG(INTERP_ADDITIONAL_CXX_FLAGS -Wno-unused-variable)
 ROOT_ADD_CXX_FLAG(INTERP_ADDITIONAL_CXX_FLAGS -Wno-unused-but-set-variable)
 ROOT_ADD_CXX_FLAG(INTERP_ADDITIONAL_CXX_FLAGS -Wno-unused-function)
+ROOT_ADD_CXX_FLAG(INTERP_ADDITIONAL_CXX_FLAGS -Wno-range-loop-analysis)
 
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${INTERP_ADDITIONAL_CXX_FLAGS}")
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${INTERP_ADDITIONAL_C_FLAGS}")
diff --git a/math/matrix/src/TVectorT.cxx b/math/matrix/src/TVectorT.cxx
index 6c8ea63ad04..3e3210aa1ec 100644
--- a/math/matrix/src/TVectorT.cxx
+++ b/math/matrix/src/TVectorT.cxx
@@ -260,7 +260,7 @@ TVectorT<Element>::TVectorT(const TMatrixTDiag_const<Element> &md) : TObject()
 /// TVectorT foo(1,3,0.0,1.0,1.5,"END");
 
 template<class Element>
-TVectorT<Element>::TVectorT(Int_t lwb,Int_t upb,Element va_(iv1), ...)
+TVectorT<Element>::TVectorT(Int_t lwb,Int_t upb,Element iv1, ...)
 {
    if (upb < lwb) {
       Error("TVectorT(Int_t, Int_t, ...)","upb(%d) < lwb(%d)",upb,lwb);
diff --git a/roofit/roofitcore/src/BidirMMapPipe.cxx b/roofit/roofitcore/src/BidirMMapPipe.cxx
index 395ecf48fb2..0729b8c96dc 100644
--- a/roofit/roofitcore/src/BidirMMapPipe.cxx
+++ b/roofit/roofitcore/src/BidirMMapPipe.cxx
@@ -114,9 +114,8 @@ namespace BidirMMapPipe_impl {
             unsigned short m_pos;       ///< index of next byte in payload area
             /// copy construction forbidden
             Page(const Page&) {}
-            /// assertignment forbidden
-            Page& operator=(const Page&)
-            { return *reinterpret_cast<Page*>(0); }
+            /// assigment forbidden
+            Page& operator=(const Page&) = delete;
         public:
             /// constructor
             Page() : m_next(0), m_size(0), m_pos(0)
-- 
GitLab