grpc
/
grpc-java
mirror of https://github.com/grpc/grpc-java.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

core: Make tiny fixes to InUseStateAggregator2 and InternalSubchannel (#2466) 

- Remove unused variable terminated from TransportListener#transportTerminated
- Do not mention getLock in the javadoc of InUseStateAggregator2#handleNotInUse
This commit is contained in:
Kenji Kaneda 2016-11-30 16:03:21 -08:00 committed by Kun Zhang
parent 5269a1b969
commit 5ff16dd4c6
2 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core/src/main/java/io/grpc/internal/InUseStateAggregator2.java
View File

@ -75,8 +75,6 @@ abstract class InUseStateAggregator2<T> {
/** /**
* Called when the aggregated in-use state has changed to false, which means no object is in use. * Called when the aggregated in-use state has changed to false, which means no object is in use.
*
* <p>This method is called under the lock returned by {@link #getLock}.
*/ */
abstract void handleNotInUse(); abstract void handleNotInUse();
} }

1
core/src/main/java/io/grpc/internal/InternalSubchannel.java
View File

@ -447,7 +447,6 @@ final class InternalSubchannel implements WithLogId {
log.log(Level.FINE, "[{0}] {1} for {2} is terminated", log.log(Level.FINE, "[{0}] {1} for {2} is terminated",
new Object[] {getLogId(), transport.getLogId(), address}); new Object[] {getLogId(), transport.getLogId(), address});
} }
boolean terminated = false;
handleTransportInUseState(transport, false); handleTransportInUseState(transport, false);
synchronized (lock) { synchronized (lock) {
transports.remove(transport); transports.remove(transport);