Bug 1022043 - Remove EventStates' copy constructor and assignment operator to make it trivially copyable so that it builds with clang-cl; r=bzbarsky
authorEhsan Akhgari <ehsan@mozilla.com>
Fri, 06 Jun 2014 22:09:34 -0400
changeset 206582 e2714481c419a8882a3918a04da888a1f452b297
parent 206581 299ed3d227d4429111c6f2ea316d30040f23197c
child 206583 397d00aa598fce9d34f9a37c89ccdba43c5f5bf4
push id3741
push userasasaki@mozilla.com
push dateMon, 21 Jul 2014 20:25:18 +0000
treeherdermozilla-beta@4d6f46f5af68 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbzbarsky
bugs1022043
milestone32.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1022043 - Remove EventStates' copy constructor and assignment operator to make it trivially copyable so that it builds with clang-cl; r=bzbarsky
dom/events/EventStates.h
--- a/dom/events/EventStates.h
+++ b/dom/events/EventStates.h
@@ -33,27 +33,16 @@ public:
   // In that case, we could be sure that only macros at the end were creating
   // EventStates instances with mStates set to something else than 0.
   // Unfortunately, this constructor is needed at at least two places now.
   explicit MOZ_CONSTEXPR EventStates(InternalType aStates)
     : mStates(aStates)
   {
   }
 
-  MOZ_CONSTEXPR EventStates(const EventStates& aEventStates)
-    : mStates(aEventStates.mStates)
-  {
-  }
-
-  EventStates& operator=(const EventStates& aEventStates)
-  {
-    mStates = aEventStates.mStates;
-    return *this;
-  }
-
   EventStates MOZ_CONSTEXPR operator|(const EventStates& aEventStates) const
   {
     return EventStates(mStates | aEventStates.mStates);
   }
 
   EventStates& operator|=(const EventStates& aEventStates)
   {
     mStates |= aEventStates.mStates;