Bug 1287006 - Final bustage followup. r=bustage in a still-CLOSED TREE

This commit is contained in:
Jeff Walden 2017-02-22 20:46:46 -08:00
parent a457461dd8
commit 938292cf1f
4 changed files with 4 additions and 4 deletions

View File

@ -61,7 +61,7 @@ public:
* specified).
* Returns true if the semaphore was succesfully decremented, false otherwise.
**/
bool Wait(Maybe<TimeDuration> aWaitTime = Nothing());
bool Wait(const Maybe<TimeDuration>& aWaitTime = Nothing());
/**
* Increments the current value of the semaphore.

View File

@ -109,7 +109,7 @@ CrossProcessSemaphore::~CrossProcessSemaphore()
}
bool
CrossProcessSemaphore::Wait(Maybe<TimeDuration> aWaitTime)
CrossProcessSemaphore::Wait(const Maybe<TimeDuration>& aWaitTime)
{
MOZ_ASSERT(*mRefCount > 0, "Attempting to wait on a semaphore with zero ref count");
int ret;

View File

@ -26,7 +26,7 @@ CrossProcessSemaphore::~CrossProcessSemaphore()
}
bool
CrossProcessSemaphore::Wait(Maybe<TimeDuration> aWaitTime)
CrossProcessSemaphore::Wait(const Maybe<TimeDuration>& aWaitTime)
{
MOZ_CRASH("Cross-process semaphores not allowed on this platform - woah! We should've aborted by now!");
return false;

View File

@ -47,7 +47,7 @@ CrossProcessSemaphore::~CrossProcessSemaphore()
}
bool
CrossProcessSemaphore::Wait(Maybe<TimeDuration> aWaitTime)
CrossProcessSemaphore::Wait(const Maybe<TimeDuration>& aWaitTime)
{
MOZ_ASSERT(mSemaphore, "Improper construction of semaphore.");
HRESULT hr = ::WaitForSingleObject(mSemaphore, aWaitTime.isSome() ?