Most of the functions in AbstractStub should be annotated with @CheckReturnValue

[Hzj-jie]

These functions create new instances rather than changing the internal state, e.g.
stub.withCallCredentials( ... )
is wrong. One should always use
stub = stub.withCallCredentials( ... )

grpc-java/stub/src/main/java/io/grpc/stub/AbstractStub.java

Line 48 in c90f27f

public abstract class AbstractStub<S extends AbstractStub<S>> {

[carl-mastrangelo]

Static analyzers will fail if the value is not checked, which is likely extremely high at this point. While the code is probably buggy, I don't think it can reasonably be fixed at this point.