On 13 Jan, 2014, at 2:01 PM, Chris Jones <notifications@github.com> wrote:

> In DataFormats/Common/interface/DetSet.h:
> 
> > @@ -51,7 +51,7 @@
> >      
> >      DetSet<T> & operator=(DetSet<T> && rh)  noexcept {
> >        id = rh.id;
> > -      data.swap(rh.data);
> > +      data = std::move(rh.data);
> 
> @VinInn Vincenzo, are there any other classes which are elements of DetSet which which do not have move op=? I ask since if there are you probably made a pessimistic optimization replacing a swap with a copy. I think the standard has some meta programming checks for the various move functions. Not 100% certain of that though.
and btw a vector is movable even if its elements are not movable
see http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52591 for instance
v.