Make the change on status effective.

Xudong Ma · Xudong Ma · commit ef106e059338 · 2015-08-04T18:06:25.000-07:00
diff --git a/core/src/main/java/io/grpc/transport/Http2ClientStream.java b/core/src/main/java/io/grpc/transport/Http2ClientStream.java
@@ -167,10 +167,8 @@ private static Status statusFromHttpStatus(Metadata metadata) {
     Integer httpStatus = metadata.get(HTTP2_STATUS);
     if (httpStatus != null) {
       Status status = HttpUtil.httpStatusToGrpcStatus(httpStatus);
-      if (!status.isOk()) {
-        status.augmentDescription("extracted status from HTTP :status " + httpStatus);
-      }
-      return status;
+      return status.isOk() ? status
+          : status.augmentDescription("extracted status from HTTP :status " + httpStatus);
     }
     return null;
   }

Original file line number	Diff line number	Diff line change
`@@ -167,10 +167,8 @@ private static Status statusFromHttpStatus(Metadata metadata) {`
`167`	`167`	`Integer httpStatus = metadata.get(HTTP2_STATUS);`
`168`	`168`	`if (httpStatus != null) {`
`169`	`169`	`Status status = HttpUtil.httpStatusToGrpcStatus(httpStatus);`
`170`		`- if (!status.isOk()) {`
`171`		`- status.augmentDescription("extracted status from HTTP :status " + httpStatus);`
`172`		`- }`
`173`		`- return status;`
	`170`	`+ return status.isOk() ? status`
	`171`	`+ : status.augmentDescription("extracted status from HTTP :status " + httpStatus);`
`174`	`172`	`}`
`175`	`173`	`return null;`
`176`	`174`	`}`