Skip to content

Commit

Permalink
reduce redundant judgment
Browse files Browse the repository at this point in the history
  • Loading branch information
CLFutureX authored and ejona86 committed Nov 10, 2022
1 parent eb1e5a1 commit b8f142c
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 12 deletions.
4 changes: 1 addition & 3 deletions api/src/main/java/io/grpc/LoadBalancerRegistry.java
Original file line number Diff line number Diff line change
Expand Up @@ -107,9 +107,7 @@ public static synchronized LoadBalancerRegistry getDefaultRegistry() {
instance = new LoadBalancerRegistry();
for (LoadBalancerProvider provider : providerList) {
logger.fine("Service loader found " + provider);
if (provider.isAvailable()) {
instance.addProvider(provider);
}
instance.addProvider(provider);
}
instance.refreshProviderMap();
}
Expand Down
4 changes: 1 addition & 3 deletions api/src/main/java/io/grpc/ManagedChannelRegistry.java
Original file line number Diff line number Diff line change
Expand Up @@ -106,9 +106,7 @@ public static synchronized ManagedChannelRegistry getDefaultRegistry() {
instance = new ManagedChannelRegistry();
for (ManagedChannelProvider provider : providerList) {
logger.fine("Service loader found " + provider);
if (provider.isAvailable()) {
instance.addProvider(provider);
}
instance.addProvider(provider);
}
instance.refreshProviders();
}
Expand Down
4 changes: 1 addition & 3 deletions api/src/main/java/io/grpc/NameResolverRegistry.java
Original file line number Diff line number Diff line change
Expand Up @@ -124,9 +124,7 @@ public static synchronized NameResolverRegistry getDefaultRegistry() {
instance = new NameResolverRegistry();
for (NameResolverProvider provider : providerList) {
logger.fine("Service loader found " + provider);
if (provider.isAvailable()) {
instance.addProvider(provider);
}
instance.addProvider(provider);
}
instance.refreshProviders();
}
Expand Down
4 changes: 1 addition & 3 deletions api/src/main/java/io/grpc/ServerRegistry.java
Original file line number Diff line number Diff line change
Expand Up @@ -98,9 +98,7 @@ public static synchronized ServerRegistry getDefaultRegistry() {
instance = new ServerRegistry();
for (ServerProvider provider : providerList) {
logger.fine("Service loader found " + provider);
if (provider.isAvailable()) {
instance.addProvider(provider);
}
instance.addProvider(provider);
}
instance.refreshProviders();
}
Expand Down

0 comments on commit b8f142c

Please sign in to comment.