Merge "Update to positive logic to better match the method naming"

This commit is contained in:
Chiachang Wang
2020-10-07 08:21:32 +00:00
committed by Gerrit Code Review
2 changed files with 4 additions and 4 deletions

View File

@@ -380,8 +380,8 @@ public class Vpn {
}
}
public boolean checkInterfacePresent(final Vpn vpn, final String iface) {
return vpn.jniCheck(iface) == 0;
public boolean isInterfacePresent(final Vpn vpn, final String iface) {
return vpn.jniCheck(iface) != 0;
}
}
@@ -2995,7 +2995,7 @@ public class Vpn {
checkInterruptAndDelay(false);
// Check if the interface is gone while we are waiting.
if (mDeps.checkInterfacePresent(Vpn.this, mConfig.interfaze)) {
if (!mDeps.isInterfacePresent(Vpn.this, mConfig.interfaze)) {
throw new IllegalStateException(mConfig.interfaze + " is gone");
}

View File

@@ -1263,7 +1263,7 @@ public class VpnTest {
}
@Override
public boolean checkInterfacePresent(final Vpn vpn, final String iface) {
public boolean isInterfacePresent(final Vpn vpn, final String iface) {
return true;
}
}