Merge pull request #27992 from Dr15Jones/clangFix_bqueue

Explicitly declare copy assignment for bqueue
cms-sw · Sep 15, 2019 · 241a280 · 241a280
2 parents 0b972d8 + 334cb45
commit 241a280
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/TrackingTools/PatternTools/interface/bqueue.h b/TrackingTools/PatternTools/interface/bqueue.h
@@ -138,6 +138,7 @@ namespace cmsutils {
       cp.m_size = 0;
     }
 
+    bqueue &operator=(bqueue<T> const &) = default;
     bqueue &operator=(bqueue<T> &&cp) noexcept {
       using std::swap;
       swap(m_size, cp.m_size);

diff --git a/TrackingTools/PatternTools/test/bqueue_t.cpp b/TrackingTools/PatternTools/test/bqueue_t.cpp
@@ -47,6 +47,15 @@ int main() {
   assert(cont.shared());
   assert(cont3.shared());
   verifySeq(cont3);
+  // copy assign
+  {
+    Cont cont_assign;
+    cont_assign = cont;
+    assert(cont.size() == cont_assign.size());
+    assert(cont.shared());
+    assert(cont_assign.shared());
+    verifySeq(cont_assign);
+  }
   // add
   cont.push_back(Cont::value_type(new int(10)));
   assert((*cont.back()) == 10);