mirror
/
FairEmail
mirror of https://github.com/M66B/FairEmail.git
1
0
Fork 0
Code Issues Releases Wiki Activity

Revert "Guarantee backoff" 

This reverts commit 3dfaa44e89.
This commit is contained in:
M66B 2020-09-21 07:34:07 +02:00
parent f7b043a594
commit 42d938f32e
2 changed files with 7 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
app/src/main/java/eu/faircode/email/Core.java
View File

@ -4053,7 +4053,6 @@ class Core {
static class State {
private int backoff;
private boolean backingoff = false;
private ConnectionHelper.NetworkState networkState;
private Thread thread = new Thread();
private Semaphore semaphore = new Semaphore(0);
@ -4098,19 +4097,11 @@ class Core {
return true;
}
boolean acquire(long milliseconds, boolean backoff) throws InterruptedException {
try {
backingoff = backoff;
return semaphore.tryAcquire(milliseconds, TimeUnit.MILLISECONDS);
} finally {
backingoff = false;
}
boolean acquire(long milliseconds) throws InterruptedException {
return semaphore.tryAcquire(milliseconds, TimeUnit.MILLISECONDS);
}
void error(Throwable ex) {
if (backingoff)
return;
if (ex instanceof MessagingException &&
("connection failure".equals(ex.getMessage()) ||
"Not connected".equals(ex.getMessage()) || // POP3

10
app/src/main/java/eu/faircode/email/ServiceSynchronize.java
View File

@ -899,7 +899,7 @@ public class ServiceSynchronize extends ServiceBase implements SharedPreferences
try {
long backoff = RECONNECT_BACKOFF - ago;
EntityLog.log(ServiceSynchronize.this, account.name + " reconnect backoff=" + (backoff / 1000));
state.acquire(backoff, true);
state.acquire(backoff);
} catch (InterruptedException ex) {
Log.w(account.name + " backoff " + ex.toString());
}
@ -1523,7 +1523,7 @@ public class ServiceSynchronize extends ServiceBase implements SharedPreferences
try {
wlAccount.release();
state.acquire(2 * duration, false);
state.acquire(2 * duration);
Log.i("### " + account.name + " keeping alive");
} catch (InterruptedException ex) {
EntityLog.log(this, account.name + " waited state=" + state);
@ -1638,7 +1638,7 @@ public class ServiceSynchronize extends ServiceBase implements SharedPreferences
if (cbackoff > backoff) {
try {
EntityLog.log(this, account.name + " reconnect backoff=" + cbackoff);
state.acquire(cbackoff * 1000L, true);
state.acquire(cbackoff * 1000L);
} catch (InterruptedException ex) {
Log.w(account.name + " cbackoff " + ex.toString());
}
@ -1648,7 +1648,7 @@ public class ServiceSynchronize extends ServiceBase implements SharedPreferences
if (backoff <= CONNECT_BACKOFF_MAX) {
// Short back-off period, keep device awake
try {
state.acquire(backoff * 1000L, true);
state.acquire(backoff * 1000L);
} catch (InterruptedException ex) {
Log.w(account.name + " backoff " + ex.toString());
}
@ -1681,7 +1681,7 @@ public class ServiceSynchronize extends ServiceBase implements SharedPreferences
try {
wlAccount.release();
state.acquire(2 * backoff * 1000L, true);
state.acquire(2 * backoff * 1000L);
Log.i("### " + account.name + " backoff done");
} catch (InterruptedException ex) {
Log.w(account.name + " backoff " + ex.toString());