Merge pull request #715 from andrewgaul/findbugs-infinite-loop

Fix infinite loops, found by FindBugs
This commit is contained in:
Adrian Cole 2012-07-14 19:14:01 -07:00
commit c8c2d5f175
2 changed files with 2 additions and 2 deletions

View File

@ -158,7 +158,7 @@ public class RetryingCacheLoaderDecorator<K, V> {
@Override @Override
protected Objects.ToStringHelper string() { protected Objects.ToStringHelper string() {
return string().add("retryableThrowable", retryableThrowable).add("periodMs", periodMs).add("maxPeriodMs", return Objects.toStringHelper(this).add("retryableThrowable", retryableThrowable).add("periodMs", periodMs).add("maxPeriodMs",
maxPeriodMs).add("maxTries", maxTries); maxPeriodMs).add("maxTries", maxTries);
} }
} }

View File

@ -264,7 +264,7 @@ public class QueryResultNetworkRecord extends QueryResultRecordType {
@Override @Override
public ToStringHelper string() { public ToStringHelper string() {
return string().add("name", name).add("ipScopeId", ipScopeId).add("gateway", gateway).add("netmask", netmask).add("dns1", dns1).add("dns2", dns2).add("dnsSuffix", dnsSuffix).add("isBusy", return Objects.toStringHelper(this).add("name", name).add("ipScopeId", ipScopeId).add("gateway", gateway).add("netmask", netmask).add("dns1", dns1).add("dns2", dns2).add("dnsSuffix", dnsSuffix).add("isBusy",
isBusy); isBusy);
} }