Bug 818817 - Fix some compiler warnings, r=Ms2ger

accessible/src/base/AccCollector.h

@@ -74,7 +74,7 @@ private:
 class EmbeddedObjCollector : public AccCollector
 {
 public:
-  virtual ~EmbeddedObjCollector() { };
+  virtual ~EmbeddedObjCollector() { }
 
 public:
   virtual int32_t GetIndexAt(Accessible* aAccessible);

accessible/src/base/AccEvent.h

@@ -213,7 +213,7 @@ public:
     mNode = aTargetNode;
     mParent = mAccessible->Parent();
   }
-  virtual ~AccMutationEvent() { };
+  virtual ~AccMutationEvent() { }
 
   // Event
   static const EventGroup kEventGroup = eMutationEvent;
@@ -291,7 +291,7 @@ public:
   AccReorderEvent(Accessible* aTarget) :
     AccEvent(::nsIAccessibleEvent::EVENT_REORDER, aTarget,
              eAutoDetect, eCoalesceReorder) { }
-  virtual ~AccReorderEvent() { };
+  virtual ~AccReorderEvent() { }
 
   // Event
   static const EventGroup kEventGroup = eReorderEvent;

accessible/src/base/NotificationController.h

@@ -28,7 +28,7 @@ class DocAccessible;
 class Notification
 {
 public:
-  virtual ~Notification() { };
+  virtual ~Notification() { }
 
   NS_INLINE_DECL_REFCOUNTING(Notification)
 

accessible/src/base/StyleInfo.h

@@ -17,7 +17,7 @@ class StyleInfo
 {
 public:
   StyleInfo(dom::Element* aElement, nsIPresShell* aPresShell);
-  ~StyleInfo() { };
+  ~StyleInfo() { }
 
   void Display(nsAString& aValue);
   void TextAlign(nsAString& aValue);

accessible/src/base/nsCoreUtils.h

@@ -304,8 +304,8 @@ public:
 class nsAccessibleDOMStringList : public nsIDOMDOMStringList
 {
 public:
-  nsAccessibleDOMStringList() {};
+  nsAccessibleDOMStringList() {}
-  virtual ~nsAccessibleDOMStringList() {};
+  virtual ~nsAccessibleDOMStringList() {}
 
   NS_DECL_ISUPPORTS
   NS_DECL_NSIDOMDOMSTRINGLIST

accessible/src/generic/Accessible.h

@@ -950,7 +950,7 @@ public:
 
   KeyBinding() : mKey(0), mModifierMask(0) {}
   KeyBinding(uint32_t aKey, uint32_t aModifierMask) :
-    mKey(aKey), mModifierMask(aModifierMask) {};
+    mKey(aKey), mModifierMask(aModifierMask) {}
 
   inline bool IsEmpty() const { return !mKey; }
   inline uint32_t Key() const { return mKey; }

accessible/src/html/HTMLElementAccessibles.h

@@ -20,7 +20,7 @@ class HTMLHRAccessible : public LeafAccessible
 public:
 
   HTMLHRAccessible(nsIContent* aContent, DocAccessible* aDoc) :
-    LeafAccessible(aContent, aDoc) {};
+    LeafAccessible(aContent, aDoc) {}
 
   // Accessible
   virtual a11y::role NativeRole();
@@ -33,7 +33,7 @@ class HTMLBRAccessible : public LeafAccessible
 {
 public:
   HTMLBRAccessible(nsIContent* aContent, DocAccessible* aDoc) :
-    LeafAccessible(aContent, aDoc) {};
+    LeafAccessible(aContent, aDoc) {}
 
   // Accessible
   virtual a11y::role NativeRole();
@@ -52,7 +52,7 @@ class HTMLLabelAccessible : public HyperTextAccessibleWrap
 public:
 
   HTMLLabelAccessible(nsIContent* aContent, DocAccessible* aDoc) :
-    HyperTextAccessibleWrap(aContent, aDoc) {};
+    HyperTextAccessibleWrap(aContent, aDoc) {}
 
   NS_DECL_ISUPPORTS_INHERITED
 
@@ -71,7 +71,7 @@ class HTMLOutputAccessible : public HyperTextAccessibleWrap
 public:
 
   HTMLOutputAccessible(nsIContent* aContent, DocAccessible* aDoc) :
-    HyperTextAccessibleWrap(aContent, aDoc) {};
+    HyperTextAccessibleWrap(aContent, aDoc) {}
 
   NS_DECL_ISUPPORTS_INHERITED
 

content/base/src/nsObjectLoadingContent.h

@@ -131,7 +131,7 @@ class nsObjectLoadingContent : public nsImageLoadingContent
      * Used by pluginHost to know if we're loading with a channel, so it
      * will not open its own.
      */
-    bool SrcStreamLoading() { return mSrcStreamLoading; };
+    bool SrcStreamLoading() { return mSrcStreamLoading; }
 
   protected:
     /**

content/media/webrtc/MediaEngine.h

@@ -31,7 +31,7 @@ enum MediaEngineState {
 class MediaEngine
 {
 public:
-  virtual ~MediaEngine() {};
+  virtual ~MediaEngine() {}
 
   /* Populate an array of video sources in the nsTArray. Also include devices
    * that are currently unavailable. */
@@ -48,7 +48,7 @@ public:
 class MediaEngineSource : public nsISupports
 {
 public:
-  virtual ~MediaEngineSource() {};
+  virtual ~MediaEngineSource() {}
 
   /* Populate the human readable name of this device in the nsAString */
   virtual void GetName(nsAString&) = 0;
@@ -114,7 +114,7 @@ struct MediaEngineVideoOptions {
 class MediaEngineVideoSource : public MediaEngineSource
 {
 public:
-  virtual ~MediaEngineVideoSource() {};
+  virtual ~MediaEngineVideoSource() {}
 
   /* Return a MediaEngineVideoOptions struct with appropriate values for all
    * fields. */
@@ -127,7 +127,7 @@ public:
 class MediaEngineAudioSource : public MediaEngineSource
 {
 public:
-  virtual ~MediaEngineAudioSource() {};
+  virtual ~MediaEngineAudioSource() {}
 };
 
 }

content/svg/content/src/DOMSVGLength.h

@@ -95,7 +95,7 @@ public:
     if (mList) {
       mList->mItems[mListIndex] = nullptr;
     }
-  };
+  }
 
   /**
    * Create an unowned copy of an owned length. The caller is responsible for

content/svg/content/src/DOMSVGLengthList.h

@@ -71,7 +71,7 @@ public:
     if (mAList) {
       ( IsAnimValList() ? mAList->mAnimVal : mAList->mBaseVal ) = nullptr;
     }
-  };
+  }
 
   virtual JSObject* WrapObject(JSContext *cx, JSObject *scope,
                                bool *triedToWrap);

content/svg/content/src/DOMSVGNumber.h

@@ -70,7 +70,7 @@ public:
     if (mList) {
       mList->mItems[mListIndex] = nullptr;
     }
-  };
+  }
 
   /**
    * Create an unowned copy. The caller is responsible for the first AddRef().

content/svg/content/src/DOMSVGTransform.h

@@ -81,7 +81,7 @@ public:
     if (mList) {
       mList->mItems[mListIndex] = nullptr;
     }
-  };
+  }
 
   /**
    * Create an unowned copy of an owned transform. The caller is responsible for

content/xul/content/src/nsXULElement.cpp

@@ -141,7 +141,7 @@ public:
     NS_ADDREF(*aStyle);
     return NS_OK;
   }
-  NS_FORWARD_NSIFRAMELOADEROWNER(static_cast<nsXULElement*>(mElement.get())->);
+  NS_FORWARD_NSIFRAMELOADEROWNER(static_cast<nsXULElement*>(mElement.get())->)
 private:
   nsCOMPtr<nsIDOMXULElement> mElement;
 };

dom/base/nsJSEnvironment.h

@@ -103,7 +103,7 @@ public:
                                    JSObject** aFunctionObject);
 
   virtual nsIScriptGlobalObject *GetGlobalObject();
-  inline nsIScriptGlobalObject *GetGlobalObjectRef() { return mGlobalObjectRef; };
+  inline nsIScriptGlobalObject *GetGlobalObjectRef() { return mGlobalObjectRef; }
 
   virtual JSContext* GetNativeContext();
   virtual JSObject* GetNativeGlobal();

dom/media/MediaManager.h

@@ -254,13 +254,13 @@ public:
     mSource = aSource;
     mType.Assign(NS_LITERAL_STRING("video"));
     mSource->GetName(mName);
-  };
+  }
   MediaDevice(MediaEngineAudioSource* aSource) {
     mSource = aSource;
     mType.Assign(NS_LITERAL_STRING("audio"));
     mSource->GetName(mName);
-  };
+  }
-  virtual ~MediaDevice() {};
+  virtual ~MediaDevice() {}
 
   MediaEngineSource* GetSource();
 private:
@@ -317,7 +317,7 @@ private:
   WindowTable *GetActiveWindows() {
     NS_ASSERTION(NS_IsMainThread(), "Only access windowlist on main thread");
     return &mActiveWindows;
-  };
+  }
 
   // Make private because we want only one instance of this class
   MediaManager()
@@ -326,11 +326,11 @@ private:
   , mBackend(nullptr) {
     mActiveWindows.Init();
     mActiveCallbacks.Init();
-  };
+  }
 
   ~MediaManager() {
     delete mBackend;
-  };
+  }
 
   // ONLY access from MainThread so we don't need to lock
   WindowTable mActiveWindows;

editor/libeditor/html/nsWSRunObject.h

@@ -84,7 +84,7 @@ public:
   }
 private:
   uint16_t mEnum;
-  void bool_conversion_helper() {};
+  void bool_conversion_helper() {}
 public:
   // Allow boolean conversion with no numeric conversion
   typedef void (WSType::*bool_type)();

image/src/Decoder.h

@@ -89,7 +89,7 @@ public:
   // If we're doing a "size decode", we more or less pass through the image
   // data, stopping only to scoop out the image dimensions. A size decode
   // must be enabled by SetSizeDecode() _before_calling Init().
-  bool IsSizeDecode() { return mSizeDecode; };
+  bool IsSizeDecode() { return mSizeDecode; }
   void SetSizeDecode(bool aSizeDecode)
   {
     NS_ABORT_IF_FALSE(!mInitialized, "Can't set size decode after Init()!");
@@ -104,10 +104,10 @@ public:
   uint32_t GetCompleteFrameCount() { return mInFrame ? mFrameCount - 1 : mFrameCount; }
 
   // Error tracking
-  bool HasError() { return HasDataError() || HasDecoderError(); };
+  bool HasError() { return HasDataError() || HasDecoderError(); }
-  bool HasDataError() { return mDataError; };
+  bool HasDataError() { return mDataError; }
-  bool HasDecoderError() { return NS_FAILED(mFailCode); };
+  bool HasDecoderError() { return NS_FAILED(mFailCode); }
-  nsresult GetDecoderError() { return mFailCode; };
+  nsresult GetDecoderError() { return mFailCode; }
   void PostResizeError() { PostDataError(); }
   bool GetDecodeDone() const {
     return mDecodeDone;

image/src/imgRequest.h

@@ -114,7 +114,7 @@ public:
   imgStatusTracker& GetStatusTracker();
 
   // Get the current principal of the image. No AddRefing.
-  inline nsIPrincipal* GetPrincipal() const { return mPrincipal.get(); };
+  inline nsIPrincipal* GetPrincipal() const { return mPrincipal.get(); }
 
   // Resize the cache entry to 0 if it exists
   void ResetCacheEntry();

image/src/imgStatusTracker.h

@@ -114,7 +114,7 @@ public:
   // with its status. Weak pointers.
   void AddConsumer(imgRequestProxy* aConsumer);
   bool RemoveConsumer(imgRequestProxy* aConsumer, nsresult aStatus);
-  size_t ConsumerCount() const { return mConsumers.Length(); };
+  size_t ConsumerCount() const { return mConsumers.Length(); }
 
   // This is intentionally non-general because its sole purpose is to support an
   // some obscure network priority logic in imgRequest. That stuff could probably
@@ -190,8 +190,8 @@ public:
   void ClearRequest();
 
   // Weak pointer getters - no AddRefs.
-  inline mozilla::image::Image* GetImage() const { return mImage; };
+  inline mozilla::image::Image* GetImage() const { return mImage; }
-  inline imgRequest* GetRequest() const { return mRequest; };
+  inline imgRequest* GetRequest() const { return mRequest; }
 
   inline imgIDecoderObserver* GetDecoderObserver() { return mTrackerObserver.get(); }
 

js/src/ctypes/CTypes.h

@@ -304,7 +304,7 @@ struct ClosureInfo
       ffi_closure_free(closure);
     if (errResult)
       js_free(errResult);
-  };
+  }
 };
 
 bool IsCTypesGlobal(JSObject* obj);

js/src/ion/Bailouts.h

@@ -135,7 +135,7 @@ class BailoutClosure
 
     void constructFrame() {
         guards_.construct();
-    };
+    }
     InvokeArgsGuard *argsGuard() {
         return &guards_.ref().iag;
     }

js/src/ion/BitSet.h

@@ -26,7 +26,7 @@ class BitSet : private TempObject
   private:
     BitSet(unsigned int max) :
         max_(max),
-        bits_(NULL) {};
+        bits_(NULL) {}
 
     unsigned int max_;
     uint32_t *bits_;

js/src/ion/FixedList.h

@@ -53,7 +53,7 @@ class FixedList
     const T &operator [](size_t index) const {
         JS_ASSERT(index < length_);
         return list_[index];
-    };
+    }
 };
 
 } // namespace ion

js/src/ion/LIR.h

@@ -624,7 +624,7 @@ class LInstruction
 
     virtual bool isCall() const {
         return false;
-    };
+    }
     uint32_t id() const {
         return id_;
     }

js/src/ion/LinearScan.h

@@ -125,8 +125,8 @@ class LinearScanAllocator : public LiveRangeAllocator<LinearScanVirtualRegister>
     void validateIntervals();
     void validateAllocations();
 #else
-    inline void validateIntervals() { };
+    inline void validateIntervals() { }
-    inline void validateAllocations() { };
+    inline void validateAllocations() { }
 #endif
 
 #ifdef JS_NUNBOX32

js/src/ion/VMFunctions.h

@@ -343,31 +343,31 @@ struct FunctionInfo<R (*)(JSContext *)> : public VMFunction {
 template <class R, class A1>
 struct FunctionInfo<R (*)(JSContext *, A1)> : public VMFunction {
     typedef R (*pf)(JSContext *, A1);
-    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_1);
+    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_1)
 };
 
 template <class R, class A1, class A2>
 struct FunctionInfo<R (*)(JSContext *, A1, A2)> : public VMFunction {
     typedef R (*pf)(JSContext *, A1, A2);
-    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_2);
+    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_2)
 };
 
 template <class R, class A1, class A2, class A3>
 struct FunctionInfo<R (*)(JSContext *, A1, A2, A3)> : public VMFunction {
     typedef R (*pf)(JSContext *, A1, A2, A3);
-    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_3);
+    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_3)
 };
 
 template <class R, class A1, class A2, class A3, class A4>
 struct FunctionInfo<R (*)(JSContext *, A1, A2, A3, A4)> : public VMFunction {
     typedef R (*pf)(JSContext *, A1, A2, A3, A4);
-    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_4);
+    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_4)
 };
 
 template <class R, class A1, class A2, class A3, class A4, class A5>
     struct FunctionInfo<R (*)(JSContext *, A1, A2, A3, A4, A5)> : public VMFunction {
     typedef R (*pf)(JSContext *, A1, A2, A3, A4, A5);
-    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_5);
+    FUNCTION_INFO_STRUCT_BODY(FOR_EACH_ARGS_5)
 };
 
 #undef FUNCTION_INFO_STRUCT_BODY

js/src/ion/shared/LIR-x86-shared.h

@@ -14,7 +14,7 @@ namespace ion {
 class LDivI : public LBinaryMath<1>
 {
   public:
-    LIR_HEADER(DivI);
+    LIR_HEADER(DivI)
 
     LDivI(const LAllocation &lhs, const LAllocation &rhs, const LDefinition &temp) {
         setOperand(0, lhs);
@@ -33,7 +33,7 @@ class LDivI : public LBinaryMath<1>
 class LModI : public LBinaryMath<1>
 {
   public:
-    LIR_HEADER(ModI);
+    LIR_HEADER(ModI)
 
     LModI(const LAllocation &lhs, const LAllocation &rhs) {
         setOperand(0, lhs);
@@ -50,7 +50,7 @@ class LModPowTwoI : public LInstructionHelper<1,1,0>
     const int32_t shift_;
 
   public:
-    LIR_HEADER(ModPowTwoI);
+    LIR_HEADER(ModPowTwoI)
 
     LModPowTwoI(const LAllocation &lhs, int32_t shift)
       : shift_(shift)
@@ -70,7 +70,7 @@ class LModPowTwoI : public LInstructionHelper<1,1,0>
 class LPowHalfD : public LInstructionHelper<1, 1, 1>
 {
   public:
-    LIR_HEADER(PowHalfD);
+    LIR_HEADER(PowHalfD)
     LPowHalfD(const LAllocation &input, const LDefinition &temp) {
         setOperand(0, input);
         setTemp(0, temp);
@@ -91,7 +91,7 @@ class LPowHalfD : public LInstructionHelper<1, 1, 1>
 class LTableSwitch : public LInstructionHelper<0, 1, 2>
 {
   public:
-    LIR_HEADER(TableSwitch);
+    LIR_HEADER(TableSwitch)
 
     LTableSwitch(const LAllocation &in, const LDefinition &inputCopy,
                  const LDefinition &jumpTablePointer, MTableSwitch *ins)
@@ -121,7 +121,7 @@ class LTableSwitch : public LInstructionHelper<0, 1, 2>
 class LTableSwitchV : public LInstructionHelper<0, BOX_PIECES, 3>
 {
   public:
-    LIR_HEADER(TableSwitchV);
+    LIR_HEADER(TableSwitchV)
 
     LTableSwitchV(const LDefinition &inputCopy, const LDefinition &floatCopy,
                   const LDefinition &jumpTablePointer, MTableSwitch *ins)
@@ -153,7 +153,7 @@ class LTableSwitchV : public LInstructionHelper<0, BOX_PIECES, 3>
 class LGuardShape : public LInstructionHelper<0, 1, 0>
 {
   public:
-    LIR_HEADER(GuardShape);
+    LIR_HEADER(GuardShape)
 
     LGuardShape(const LAllocation &in) {
         setOperand(0, in);
@@ -166,7 +166,7 @@ class LGuardShape : public LInstructionHelper<0, 1, 0>
 class LRecompileCheck : public LInstructionHelper<0, 0, 0>
 {
   public:
-    LIR_HEADER(RecompileCheck);
+    LIR_HEADER(RecompileCheck)
 
     const MRecompileCheck *mir() const {
         return mir_->toRecompileCheck();
@@ -176,13 +176,13 @@ class LRecompileCheck : public LInstructionHelper<0, 0, 0>
 class LInterruptCheck : public LInstructionHelper<0, 0, 0>
 {
   public:
-    LIR_HEADER(InterruptCheck);
+    LIR_HEADER(InterruptCheck)
 };
 
 class LMulI : public LBinaryMath<0, 1>
 {
   public:
-    LIR_HEADER(MulI);
+    LIR_HEADER(MulI)
 
     LMulI(const LAllocation &lhs, const LAllocation &rhs, const LAllocation &lhsCopy) {
         setOperand(0, lhs);

js/src/ion/x64/LIR-x64.h

@@ -19,7 +19,7 @@ class LBox : public LInstructionHelper<1, 1, 0>
     MIRType type_;
 
   public:
-    LIR_HEADER(Box);
+    LIR_HEADER(Box)
 
     LBox(MIRType type, const LAllocation &payload)
       : type_(type)
@@ -50,7 +50,7 @@ class LUnboxBase : public LInstructionHelper<1, 1, 0>
 
 class LUnbox : public LUnboxBase {
   public:
-    LIR_HEADER(Unbox);
+    LIR_HEADER(Unbox)
 
     LUnbox(const LAllocation &input)
       : LUnboxBase(input)
@@ -59,7 +59,7 @@ class LUnbox : public LUnboxBase {
 
 class LUnboxDouble : public LUnboxBase {
   public:
-    LIR_HEADER(UnboxDouble);
+    LIR_HEADER(UnboxDouble)
 
     LUnboxDouble(const LAllocation &input)
       : LUnboxBase(input)
@@ -72,7 +72,7 @@ class LDouble : public LInstructionHelper<1, 0, 0>
     double d_;
 
   public:
-    LIR_HEADER(Double);
+    LIR_HEADER(Double)
 
     LDouble(double d)
       : d_(d)

js/src/jsinfer.h

@@ -51,7 +51,7 @@ class TaggedProto
 template <>
 struct RootKind<TaggedProto>
 {
-    static ThingRootKind rootKind() { return THING_ROOT_OBJECT; };
+    static ThingRootKind rootKind() { return THING_ROOT_OBJECT; }
 };
 
 template <> struct RootMethods<const TaggedProto>

js/src/jswrapper.h

@@ -49,8 +49,8 @@ class JS_FRIEND_API(Wrapper) : public DirectProxyHandler
      * is not safe to unwrap, operations requiring full access to the underlying
      * object (via UnwrapObjectChecked) will throw. Otherwise, they will succeed.
      */
-    void setSafeToUnwrap(bool safe) { mSafeToUnwrap = safe; };
+    void setSafeToUnwrap(bool safe) { mSafeToUnwrap = safe; }
-    bool isSafeToUnwrap() { return mSafeToUnwrap; };
+    bool isSafeToUnwrap() { return mSafeToUnwrap; }
 
     static JSObject *New(JSContext *cx, JSObject *obj, JSObject *proto,
                          JSObject *parent, Wrapper *handler);

js/src/methodjit/FrameState.h

@@ -826,7 +826,7 @@ class FrameState
 #ifdef DEBUG
     void assertValidRegisterState() const;
 #else
-    inline void assertValidRegisterState() const {};
+    inline void assertValidRegisterState() const {}
 #endif
 
     // Return an address, relative to the StackFrame, that represents where

js/xpconnect/wrappers/ChromeObjectWrapper.h

@@ -24,7 +24,7 @@ namespace xpc {
 class ChromeObjectWrapper : public ChromeObjectWrapperBase
 {
   public:
-    ChromeObjectWrapper() : ChromeObjectWrapperBase(0) {};
+    ChromeObjectWrapper() : ChromeObjectWrapperBase(0) {}
 
     /* Custom traps. */
     virtual bool getPropertyDescriptor(JSContext *cx, JSObject *wrapper,

layout/mathml/nsMathMLmactionFrame.h

@@ -66,7 +66,7 @@ private:
     NS_DECL_ISUPPORTS
     NS_DECL_NSIDOMEVENTLISTENER
 
-    MouseListener(nsMathMLmactionFrame* aOwner) : mOwner(aOwner) { };
+    MouseListener(nsMathMLmactionFrame* aOwner) : mOwner(aOwner) { }
 
     nsMathMLmactionFrame* mOwner;
   };

layout/xul/base/src/nsMenuFrame.h

@@ -240,7 +240,7 @@ protected:
   NS_IMETHOD AttributeChanged(int32_t aNameSpaceID,
                               nsIAtom* aAttribute,
                               int32_t aModType) MOZ_OVERRIDE;
-  virtual ~nsMenuFrame() { };
+  virtual ~nsMenuFrame() { }
 
   bool SizeToPopup(nsBoxLayoutState& aState, nsSize& aSize);
 

media/mtransport/test/sockettransportservice_unittest.cpp

@@ -79,7 +79,7 @@ class SocketTransportServiceTest : public ::testing::Test {
 class EventReceived : public nsRunnable {
 public:
   EventReceived(SocketTransportServiceTest *test) :
-      test_(test) {};
+      test_(test) {}
 
   NS_IMETHOD Run() {
     test_->ReceiveEvent();
@@ -94,7 +94,7 @@ public:
 class RegisterEvent : public nsRunnable {
 public:
   RegisterEvent(SocketTransportServiceTest *test) :
-      test_(test) {};
+      test_(test) {}
 
   NS_IMETHOD Run() {
     test_->RegisterHandler();

media/mtransport/test/transport_unittests.cpp

@@ -74,7 +74,7 @@ class TransportLayerLossy : public TransportLayer {
     SignalPacketReceived(this, data, len);
   }
 
-  TRANSPORT_LAYER_ID("lossy");
+  TRANSPORT_LAYER_ID("lossy")
 
  protected:
   virtual void WasInserted() {

media/webrtc/signaling/include/CC_Call.h

@@ -20,7 +20,7 @@ namespace CSF
         CC_Call () { }
 
     public:
-        virtual ~CC_Call () {};
+        virtual ~CC_Call () {}
 
 		virtual void setRemoteWindow (VideoWindowHandle window) = 0;
 

media/webrtc/signaling/include/CC_Device.h

@@ -20,7 +20,7 @@ namespace CSF
         CC_Device() {}
 
     public:
-        virtual ~CC_Device() {};
+        virtual ~CC_Device() {}
 
         virtual std::string toString() = 0;
 

media/webrtc/signaling/src/common/NullTransport.h

@@ -32,7 +32,7 @@ public:
 
   NullTransport() {}
 
-  virtual ~NullTransport() {};
+  virtual ~NullTransport() {}
 
 private:
   NullTransport(const NullTransport& other) MOZ_DELETE;

media/webrtc/signaling/src/media-conduit/MediaConduitInterface.h

@@ -26,7 +26,7 @@ namespace mozilla {
 class TransportInterface
 {
 public:
-  virtual ~TransportInterface() {};
+  virtual ~TransportInterface() {}
 
   /**
    * RTP Transport Function to be implemented by concrete transport implementation
@@ -58,7 +58,7 @@ public:
 class VideoRenderer
 {
  public:
-  virtual ~VideoRenderer() {} ;
+  virtual ~VideoRenderer() {}
 
   /**
    * Callback Function reportng any change in the video-frame dimensions
@@ -106,7 +106,7 @@ class MediaSessionConduit
 public:
   enum Type { AUDIO, VIDEO } ;
 
-  virtual ~MediaSessionConduit() {};
+  virtual ~MediaSessionConduit() {}
 
   virtual Type type() const = 0;
 
@@ -159,7 +159,7 @@ public:
    */
   static RefPtr<VideoSessionConduit> Create();
 
-  virtual ~VideoSessionConduit() {};
+  virtual ~VideoSessionConduit() {}
 
   virtual Type type() const { return VIDEO; }
 
@@ -228,7 +228,7 @@ public:
     */
   static mozilla::RefPtr<AudioSessionConduit> Create();
 
-  virtual ~AudioSessionConduit() {};
+  virtual ~AudioSessionConduit() {}
 
   virtual Type type() const { return AUDIO; }
 

media/webrtc/trunk/src/voice_engine/include/voe_volume_control.h

@@ -118,8 +118,8 @@ public:
     virtual int GetOutputVolumePan(int channel, float& left, float& right) = 0;
 
 protected:
-    VoEVolumeControl() {};
+    VoEVolumeControl() {}
-    virtual ~VoEVolumeControl() {};
+    virtual ~VoEVolumeControl() {}
 };
 
 }  // namespace webrtc

netwerk/streamconv/test/TestStreamConv.cpp

@@ -55,7 +55,7 @@ public:
     // nsISupports declaration
     NS_DECL_ISUPPORTS
 
-    EndListener() {};
+    EndListener() {}
 
     // nsIStreamListener method
     NS_IMETHOD OnDataAvailable(nsIRequest* request, nsISupports *ctxt, nsIInputStream *inStr, 

netwerk/test/TestCallbacks.cpp

@@ -53,7 +53,7 @@ public:
         *_retval = true;
         if (aPtr != this) *_retval = false;
         return NS_OK;
-    };
+    }
 };
 
 NS_IMPL_THREADSAFE_ISUPPORTS1(ConsumerContext, nsIEquals)

security/manager/boot/src/nsSecureBrowserUIImpl.h

@@ -60,7 +60,7 @@ public:
   NS_IMETHOD Notify(nsIDOMHTMLFormElement* formNode, nsIDOMWindow* window,
                     nsIURI *actionURL, bool* cancelSubmit);
   NS_IMETHOD NotifyInvalidSubmit(nsIDOMHTMLFormElement* formNode,
-                                 nsIArray* invalidElements) { return NS_OK; };
+                                 nsIArray* invalidElements) { return NS_OK; }
   
 protected:
   mozilla::ReentrantMonitor mReentrantMonitor;

security/manager/ssl/src/nsCertificatePrincipal.h

@@ -26,7 +26,7 @@ public:
                          , mCert(aCert)
     {}
 
-  virtual ~nsCertificatePrincipal() {};
+  virtual ~nsCertificatePrincipal() {}
 
 private:
   nsCString             mFingerprint;

toolkit/components/mediasniffer/nsMediaSniffer.h

@@ -26,7 +26,7 @@ class nsMediaSniffer MOZ_FINAL : public nsIContentSniffer
     NS_DECL_ISUPPORTS
     NS_DECL_NSICONTENTSNIFFER
   protected:
-    ~nsMediaSniffer() {};
+    ~nsMediaSniffer() {}
 
 #define PATTERN_ENTRY(mask, pattern, contentType) \
     {(const uint8_t*)mask, (const uint8_t*)pattern, sizeof(mask) - 1, contentType}

toolkit/components/url-classifier/Classifier.h

@@ -58,9 +58,9 @@ public:
    */
   nsresult MarkSpoiled(nsTArray<nsCString>& aTables);
   nsresult CacheCompletions(const CacheResultArray& aResults);
-  uint32_t GetHashKey(void) { return mHashKey; };
+  uint32_t GetHashKey(void) { return mHashKey; }
-  void SetFreshTime(uint32_t aTime) { mFreshTime = aTime; };
+  void SetFreshTime(uint32_t aTime) { mFreshTime = aTime; }
-  void SetPerClientRandomize(bool aRandomize) { mPerClientRandomize = aRandomize; };
+  void SetPerClientRandomize(bool aRandomize) { mPerClientRandomize = aRandomize; }
   /*
    * Get a bunch of extra prefixes to query for completion
    * and mask the real entry being requested

toolkit/components/url-classifier/HashStore.h

@@ -45,8 +45,8 @@ public:
   void NewAddComplete(uint32_t aChunk, const Completion& aCompletion);
   void NewSubComplete(uint32_t aAddChunk, const Completion& aCompletion,
                       uint32_t aSubChunk);
-  void SetLocalUpdate(void) { mLocalUpdate = true; };
+  void SetLocalUpdate(void) { mLocalUpdate = true; }
-  bool IsLocalUpdate(void) { return mLocalUpdate; };
+  bool IsLocalUpdate(void) { return mLocalUpdate; }
 
   ChunkSet& AddChunks() { return mAddChunks; }
   ChunkSet& SubChunks() { return mSubChunks; }
@@ -79,7 +79,7 @@ public:
   HashStore(const nsACString& aTableName, nsIFile* aStoreFile);
   ~HashStore();
 
-  const nsCString& TableName() const { return mTableName; };
+  const nsCString& TableName() const { return mTableName; }
 
   nsresult Open();
   nsresult AugmentAdds(const nsTArray<uint32_t>& aPrefixes);

toolkit/components/url-classifier/nsUrlClassifierPrefixSet.cpp

@@ -37,7 +37,7 @@ class nsPrefixSetReporter : public nsIMemoryReporter
 {
 public:
   nsPrefixSetReporter(nsUrlClassifierPrefixSet* aParent, const nsACString& aName);
-  virtual ~nsPrefixSetReporter() {};
+  virtual ~nsPrefixSetReporter() {}
 
   NS_DECL_ISUPPORTS
   NS_DECL_NSIMEMORYREPORTER

toolkit/system/gnome/nsGSettingsService.cpp

@@ -84,7 +84,7 @@ public:
   NS_DECL_NSIGSETTINGSCOLLECTION
 
   nsGSettingsCollection(GSettings* aSettings) : mSettings(aSettings),
-                                                mKeys(NULL) {};
+                                                mKeys(NULL) {}
   ~nsGSettingsCollection();
 
 private:

xpcom/tests/TestAutoPtr.cpp

@@ -12,14 +12,14 @@
 class TestObjectBaseA {
     public:
         // Virtual dtor for deleting through base class pointer
-        virtual ~TestObjectBaseA() { };
+        virtual ~TestObjectBaseA() { }
         int fooA;
 };
 
 class TestObjectBaseB {
     public:
         // Virtual dtor for deleting through base class pointer
-        virtual ~TestObjectBaseB() { };
+        virtual ~TestObjectBaseB() { }
         int fooB;
 };
 

xpcom/tests/TestHashtables.cpp

@@ -72,7 +72,7 @@ public:
 
   EntityToUnicodeEntry(const char* aKey) { mNode = nullptr; }
   EntityToUnicodeEntry(const EntityToUnicodeEntry& aEntry) { mNode = aEntry.mNode; }
-  ~EntityToUnicodeEntry() { };
+  ~EntityToUnicodeEntry() { }
 
   bool KeyEquals(const char* aEntity) const { return !strcmp(mNode->mStr, aEntity); }
   static const char* KeyToPointer(const char* aEntity) { return aEntity; }

xpcom/tests/TestSettingsAPI.cpp

@@ -85,13 +85,13 @@ public:
     }
     callbackCount--;
     return NS_OK;
-  };
+  }
 
   NS_IMETHOD HandleError(const nsAString &name) {
     fprintf(stderr, "HANDLE Error! %s\n", NS_LossyConvertUTF16toASCII(name).get());
     errors++;
     return NS_OK;
-  };
+  }
 };
 
 NS_IMPL_THREADSAFE_ISUPPORTS1(SettingsServiceCallback, nsISettingsServiceCallback)