Merge "Update to positive logic to better match the method naming"
This commit is contained in:
@@ -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");
|
||||
}
|
||||
|
||||
|
||||
@@ -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;
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user