diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2016-03-04 14:21:22 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-03-04 14:21:22 +0100 |
commit | 9dcd54014c315586e8c1aae08c6079dac7acb0b6 (patch) | |
tree | 1cd0f3e0a3def08f5fdcc24937da14caaefa2cb4 | |
parent | e04284587a5db51d96430137baa76bbef3937a5d (diff) |
cid#1327083 SWL: Sleep with lock held
It's unclear to me why the disposeOffice method was marked "synchronized" in the
first place with fbd72be142ed23b3b526c8d0c3e002f15ca7ec74 "INTEGRATION: CWS
qadev16: #114831#." Maybe just some cluelessness? Lets see whether we ever run
into any problems after removing it.
Change-Id: I1f262b6b4b4f4e8911754a6b3fa3d61f5df0aeae
-rw-r--r-- | qadevOOo/runner/helper/OfficeProvider.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qadevOOo/runner/helper/OfficeProvider.java b/qadevOOo/runner/helper/OfficeProvider.java index 26d0528223e3..584c4ca48ea9 100644 --- a/qadevOOo/runner/helper/OfficeProvider.java +++ b/qadevOOo/runner/helper/OfficeProvider.java @@ -434,7 +434,7 @@ public class OfficeProvider implements AppProvider return msf; } - private synchronized boolean disposeOffice(XMultiServiceFactory msf, + private boolean disposeOffice(XMultiServiceFactory msf, TestParameters param) { XDesktop desk = null; |