no bug - remove a bunch of extra ';'s to make -pedantic happy
authorTrevor Saunders <trev.saunders@gmail.com>
Fri, 11 Jan 2013 05:14:04 -0500
changeset 124754 c911625fe20b86ad81317867ce6b2a65f2d99861
parent 124753 f52405c01335cb36a37babb2fc6dfa26988d0cab
child 124755 b91053186723750677a0e07e9b4d756075aa0f06
push id3384
push userlsblakk@mozilla.com
push dateTue, 19 Feb 2013 18:42:39 +0000
treeherdermozilla-aurora@d8c97bae8521 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
milestone21.0a1
no bug - remove a bunch of extra ';'s to make -pedantic happy
docshell/base/LoadContext.cpp
dom/base/nsContentPermissionHelper.cpp
dom/camera/CameraRecorderProfiles.h
dom/devicestorage/DeviceStorageRequestParent.cpp
dom/indexedDB/IDBDatabase.cpp
dom/indexedDB/OpenDatabaseHelper.cpp
dom/ipc/ProcessPriorityManager.cpp
dom/src/geolocation/nsGeolocation.cpp
extensions/cookie/nsPermissionManager.h
widget/windows/JumpListItem.h
widget/windows/WinTaskbar.cpp
widget/windows/nsWidgetFactory.cpp
widget/xpwidgets/GfxInfoBase.cpp
--- a/docshell/base/LoadContext.cpp
+++ b/docshell/base/LoadContext.cpp
@@ -7,17 +7,17 @@
 #include "mozilla/LoadContext.h"
 #include "nsIScriptSecurityManager.h"
 #include "nsServiceManagerUtils.h"
 #include "nsContentUtils.h"
 #include "mozIApplication.h"
 
 namespace mozilla {
 
-NS_IMPL_ISUPPORTS1(LoadContext, nsILoadContext);
+NS_IMPL_ISUPPORTS1(LoadContext, nsILoadContext)
 
 //-----------------------------------------------------------------------------
 // LoadContext::nsILoadContext
 //-----------------------------------------------------------------------------
 
 NS_IMETHODIMP
 LoadContext::GetAssociatedWindow(nsIDOMWindow**)
 {
--- a/dom/base/nsContentPermissionHelper.cpp
+++ b/dom/base/nsContentPermissionHelper.cpp
@@ -43,17 +43,17 @@ nsContentPermissionRequestProxy::Init(co
 }
 
 void
 nsContentPermissionRequestProxy::OnParentDestroyed()
 {
   mParent = nullptr;
 }
 
-NS_IMPL_ISUPPORTS1(nsContentPermissionRequestProxy, nsIContentPermissionRequest);
+NS_IMPL_ISUPPORTS1(nsContentPermissionRequestProxy, nsIContentPermissionRequest)
 
 NS_IMETHODIMP
 nsContentPermissionRequestProxy::GetType(nsACString & aType)
 {
   aType = mType;
   return NS_OK;
 }
 
--- a/dom/camera/CameraRecorderProfiles.h
+++ b/dom/camera/CameraRecorderProfiles.h
@@ -223,11 +223,11 @@ public:
 protected:
   RecorderProfileManager(uint32_t aCameraId);
   virtual ~RecorderProfileManager();
 
   uint32_t mCameraId;
   uint32_t mMaxQualityIndex;
 };
 
-}; // namespace mozilla
+} // namespace mozilla
 
 #endif // DOM_CAMERA_CAMERA_RECORDER_PROFILES_H
--- a/dom/devicestorage/DeviceStorageRequestParent.cpp
+++ b/dom/devicestorage/DeviceStorageRequestParent.cpp
@@ -213,18 +213,18 @@ DeviceStorageRequestParent::EnsureRequir
   return true;
 }
 
 DeviceStorageRequestParent::~DeviceStorageRequestParent()
 {
   MOZ_COUNT_DTOR(DeviceStorageRequestParent);
 }
 
-NS_IMPL_THREADSAFE_ADDREF(DeviceStorageRequestParent);
-NS_IMPL_THREADSAFE_RELEASE(DeviceStorageRequestParent);
+NS_IMPL_THREADSAFE_ADDREF(DeviceStorageRequestParent)
+NS_IMPL_THREADSAFE_RELEASE(DeviceStorageRequestParent)
 
 void
 DeviceStorageRequestParent::ActorDestroy(ActorDestroyReason)
 {
   MutexAutoLock lock(mMutex);
   mActorDestoryed = true;
   int32_t count = mRunnables.Length();
   for (int32_t index = 0; index < count; index++) {
--- a/dom/indexedDB/IDBDatabase.cpp
+++ b/dom/indexedDB/IDBDatabase.cpp
@@ -438,19 +438,19 @@ NS_INTERFACE_MAP_BEGIN_CYCLE_COLLECTION_
   NS_DOM_INTERFACE_MAP_ENTRY_CLASSINFO(IDBDatabase)
 NS_INTERFACE_MAP_END_INHERITING(IDBWrapperCache)
 
 NS_IMPL_ADDREF_INHERITED(IDBDatabase, IDBWrapperCache)
 NS_IMPL_RELEASE_INHERITED(IDBDatabase, IDBWrapperCache)
 
 DOMCI_DATA(IDBDatabase, IDBDatabase)
 
-NS_IMPL_EVENT_HANDLER(IDBDatabase, abort);
-NS_IMPL_EVENT_HANDLER(IDBDatabase, error);
-NS_IMPL_EVENT_HANDLER(IDBDatabase, versionchange);
+NS_IMPL_EVENT_HANDLER(IDBDatabase, abort)
+NS_IMPL_EVENT_HANDLER(IDBDatabase, error)
+NS_IMPL_EVENT_HANDLER(IDBDatabase, versionchange)
 
 NS_IMETHODIMP
 IDBDatabase::GetName(nsAString& aName)
 {
   NS_ASSERTION(NS_IsMainThread(), "Wrong thread!");
   aName.Assign(mName);
   return NS_OK;
 }
--- a/dom/indexedDB/OpenDatabaseHelper.cpp
+++ b/dom/indexedDB/OpenDatabaseHelper.cpp
@@ -1563,17 +1563,17 @@ private:
   nsRefPtr<IDBOpenDBRequest> mRequest;
   nsTArray<nsRefPtr<IDBDatabase> > mWaitingDatabases;
   int64_t mOldVersion;
   int64_t mNewVersion;
 };
 
 } // anonymous namespace
 
-NS_IMPL_THREADSAFE_ISUPPORTS1(OpenDatabaseHelper, nsIRunnable);
+NS_IMPL_THREADSAFE_ISUPPORTS1(OpenDatabaseHelper, nsIRunnable)
 
 nsresult
 OpenDatabaseHelper::Init()
 {
   mDatabaseId = IndexedDatabaseManager::GetDatabaseId(mASCIIOrigin, mName);
   NS_ENSURE_TRUE(mDatabaseId, NS_ERROR_FAILURE);
 
   return NS_OK;
@@ -2241,17 +2241,17 @@ OpenDatabaseHelper::ReleaseMainThreadObj
 
   mOpenDBRequest = nullptr;
   mDatabase = nullptr;
   mDatabaseId = nullptr;
 
   HelperBase::ReleaseMainThreadObjects();
 }
 
-NS_IMPL_ISUPPORTS_INHERITED0(SetVersionHelper, AsyncConnectionHelper);
+NS_IMPL_ISUPPORTS_INHERITED0(SetVersionHelper, AsyncConnectionHelper)
 
 nsresult
 SetVersionHelper::Init()
 {
   // Block transaction creation until we are done.
   mOpenHelper->BlockDatabase();
 
   return NS_OK;
--- a/dom/ipc/ProcessPriorityManager.cpp
+++ b/dom/ipc/ProcessPriorityManager.cpp
@@ -165,17 +165,17 @@ private:
   ProcessPriority mProcessPriority;
 
   nsTArray<nsWeakPtr> mWindows;
   nsCOMPtr<nsITimer> mGracePeriodTimer;
   nsWeakPtr mMemoryMinimizerRunnable;
   TimeStamp mStartupTime;
 };
 
-NS_IMPL_ISUPPORTS2(ProcessPriorityManager, nsIObserver, nsIDOMEventListener);
+NS_IMPL_ISUPPORTS2(ProcessPriorityManager, nsIObserver, nsIDOMEventListener)
 
 ProcessPriorityManager::ProcessPriorityManager()
   : mProcessPriority(PROCESS_PRIORITY_FOREGROUND)
   , mStartupTime(TimeStamp::Now())
 {
 }
 
 void
--- a/dom/src/geolocation/nsGeolocation.cpp
+++ b/dom/src/geolocation/nsGeolocation.cpp
@@ -116,17 +116,17 @@ public:
   {
     nsRefPtr<nsGeolocationService> gs = nsGeolocationService::GetGeolocationService();
     if (gs) {
       gs->HandleMozsettingValue(aValue);
     }
   }
 };
 
-NS_IMPL_ISUPPORTS1(GeolocationSettingsCallback, nsISettingsServiceCallback);
+NS_IMPL_ISUPPORTS1(GeolocationSettingsCallback, nsISettingsServiceCallback)
 
 class RequestPromptEvent : public nsRunnable
 {
 public:
   RequestPromptEvent(nsGeolocationRequest* request)
     : mRequest(request)
   {
   }
--- a/extensions/cookie/nsPermissionManager.h
+++ b/extensions/cookie/nsPermissionManager.h
@@ -85,17 +85,17 @@ public:
       nsAutoCString str;
       str.Assign(mHost);
       str.AppendInt(mAppId);
       str.AppendInt(static_cast<int32_t>(mIsInBrowserElement));
 
       return mozilla::HashString(str);
     }
 
-    NS_INLINE_DECL_THREADSAFE_REFCOUNTING(PermissionKey);
+    NS_INLINE_DECL_THREADSAFE_REFCOUNTING(PermissionKey)
 
     nsCString mHost;
     uint32_t  mAppId;
     bool      mIsInBrowserElement;
 
   private:
     // Default ctor shouldn't be used.
     PermissionKey() MOZ_DELETE;
--- a/widget/windows/JumpListItem.h
+++ b/widget/windows/JumpListItem.h
@@ -88,17 +88,17 @@ protected:
 class JumpListShortcut : public JumpListItem, public nsIJumpListShortcut
 {
 public:
   JumpListShortcut() :
    JumpListItem(nsIJumpListItem::JUMPLIST_ITEM_SHORTCUT)
   {}
 
   NS_DECL_CYCLE_COLLECTING_ISUPPORTS
-  NS_DECL_CYCLE_COLLECTION_CLASS_AMBIGUOUS(JumpListShortcut, JumpListItem);
+  NS_DECL_CYCLE_COLLECTION_CLASS_AMBIGUOUS(JumpListShortcut, JumpListItem)
   NS_IMETHOD GetType(int16_t *aType) { return JumpListItem::GetType(aType); }
   NS_IMETHOD Equals(nsIJumpListItem *item, bool *_retval);
   NS_DECL_NSIJUMPLISTSHORTCUT
 
   static nsresult GetShellLink(nsCOMPtr<nsIJumpListItem>& item, 
                                nsRefPtr<IShellLinkW>& aShellLink, 
                                nsCOMPtr<nsIThread> &aIOThread);
   static nsresult GetJumpListShortcut(IShellLinkW *pLink, nsCOMPtr<nsIJumpListShortcut>& aShortcut);
--- a/widget/windows/WinTaskbar.cpp
+++ b/widget/windows/WinTaskbar.cpp
@@ -187,17 +187,17 @@ DefaultController::OnActivate(bool *rAcc
   return NS_OK;
 }
 
 NS_IMETHODIMP
 DefaultController::OnClick(nsITaskbarPreviewButton *button) {
   return NS_OK;
 }
 
-NS_IMPL_ISUPPORTS1(DefaultController, nsITaskbarPreviewController);
+NS_IMPL_ISUPPORTS1(DefaultController, nsITaskbarPreviewController)
 }
 
 namespace mozilla {
 namespace widget {
 
 ///////////////////////////////////////////////////////////////////////////////
 // nsIWinTaskbar
 
--- a/widget/windows/nsWidgetFactory.cpp
+++ b/widget/windows/nsWidgetFactory.cpp
@@ -147,17 +147,17 @@ NS_GENERIC_FACTORY_CONSTRUCTOR_INIT(nsPr
 NS_GENERIC_FACTORY_CONSTRUCTOR(nsPrinterEnumeratorWin)
 NS_GENERIC_FACTORY_CONSTRUCTOR_INIT(nsPrintSession, Init)
 NS_GENERIC_FACTORY_CONSTRUCTOR(nsDeviceContextSpecWin)
 #endif
 
 namespace mozilla {
 namespace widget {
 // This constructor should really be shared with all platforms.
-NS_GENERIC_FACTORY_CONSTRUCTOR_INIT(GfxInfo, Init);
+NS_GENERIC_FACTORY_CONSTRUCTOR_INIT(GfxInfo, Init)
 }
 }
 
 NS_DEFINE_NAMED_CID(NS_WINDOW_CID);
 NS_DEFINE_NAMED_CID(NS_CHILD_CID);
 NS_DEFINE_NAMED_CID(NS_FILEPICKER_CID);
 NS_DEFINE_NAMED_CID(NS_APPSHELL_CID);
 NS_DEFINE_NAMED_CID(NS_SCREENMANAGER_CID);
--- a/widget/xpwidgets/GfxInfoBase.cpp
+++ b/widget/xpwidgets/GfxInfoBase.cpp
@@ -59,17 +59,17 @@ public:
 
     for (uint32_t i = 0; i < DeviceVendorMax; i++)
       delete GfxDriverInfo::mDeviceVendors[i];
 
     return NS_OK;
   }
 };
 
-NS_IMPL_ISUPPORTS1(ShutdownObserver, nsIObserver);
+NS_IMPL_ISUPPORTS1(ShutdownObserver, nsIObserver)
 
 void InitGfxDriverInfoShutdownObserver()
 {
   if (GfxInfoBase::mDriverInfoObserverInitialized)
     return;
 
   GfxInfoBase::mDriverInfoObserverInitialized = true;