const T& instead T (#27746)

4 years ago · 5ecbbeaa0c
parent d4f9dbefba
commit 5ecbbeaa0c
2 changed files with 2 additions and 2 deletions
--- a/src/core/lib/gprpp/status_helper.cc
+++ b/src/core/lib/gprpp/status_helper.cc
@ -331,7 +331,7 @@ std::string StatusToString(const absl::Status& status) {

 namespace internal {

-google_rpc_Status* StatusToProto(absl::Status status, upb_arena* arena) {
+google_rpc_Status* StatusToProto(const absl::Status& status, upb_arena* arena) {
  google_rpc_Status* msg = google_rpc_Status_new(arena);
  google_rpc_Status_set_code(msg, int32_t(status.code()));
  google_rpc_Status_set_message(
--- a/src/core/lib/gprpp/status_helper.h
+++ b/src/core/lib/gprpp/status_helper.h
@ -153,7 +153,7 @@ namespace internal {

 /// Builds a upb message, google_rpc_Status from a status
 /// This is for internal implementation & test only
-google_rpc_Status* StatusToProto(absl::Status status,
+google_rpc_Status* StatusToProto(const absl::Status& status,
                                 upb_arena* arena) GRPC_MUST_USE_RESULT;

 /// Builds a status from a upb message, google_rpc_Status