Author: sebb
Date: Thu Sep 20 16:02:23 2012
New Revision: 1388090
URL: http://svn.apache.org/viewvc?rev=1388090&view=rev
Log:
Override of sync. method needs to be sync.
Modified:
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
Modified:
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
URL:
http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java?rev=1388090&r1=1388089&r2=1388090&view=diff
==============================================================================
---
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
(original)
+++
commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/concurrent/TimedSemaphoreTest.java
Thu Sep 20 16:02:23 2012
@@ -405,7 +405,7 @@ public class TimedSemaphoreTest {
* Invokes the latch if one is set.
*/
@Override
- public void acquire() throws InterruptedException {
+ public synchronized void acquire() throws InterruptedException {
super.acquire();
if (latch != null) {
latch.countDown();
@@ -416,11 +416,9 @@ public class TimedSemaphoreTest {
* Counts the number of invocations.
*/
@Override
- protected void endOfPeriod() {
+ protected synchronized void endOfPeriod() {
super.endOfPeriod();
- synchronized (this) {
- periodEnds++;
- }
+ periodEnds++;
}
/**
