Mercury-IM/Smack
Mercury-IM/Smack
1
0
Fork 0
mirror of https://codeberg.org/Mercury-IM/Smack synced 2024-11-22 14:22:05 +01:00
Code Issues Releases Wiki Activity

Deprecate SynchronizationPoint.reportFailure()

Browse source 
assert and document that an exception must be set when calling
reportFailure(E).
This commit is contained in:
Florian Schmaus 2015-04-07 08:52:48 +02:00
parent dabbb40de6
commit 9a69f992c4
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
smack-core/src/main/java/org/jivesoftware/smack/SynchronizationPoint.java
View file

@ -160,16 +160,22 @@ public class SynchronizationPoint<E extends Exception> {
} }
} }
/**
* Deprecated
* @deprecated use {@link #reportFailure(Exception)} instead.
*/
@Deprecated
public void reportFailure() { public void reportFailure() {
reportFailure(null); reportFailure(null);
} }
/** /**
* Report this synchronization point as failed because of the given exception. * Report this synchronization point as failed because of the given exception. The {@code failureException} must be set.
* *
* @param failureException the exception causing this synchronization point to fail. * @param failureException the exception causing this synchronization point to fail.
*/ */
public void reportFailure(E failureException) { public void reportFailure(E failureException) {
assert failureException != null;
connectionLock.lock(); connectionLock.lock();
try { try {
state = State.Failure; state = State.Failure;