commit d1b253ab70b17abb33e5778e2c2f99b65bac8564
author rch <rch@google.com> Thu Mar 31 12:44:56 2022 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Mar 31 12:45:40 2022 -0700
tree b49251e7978458f1986ed9baf0b2f85a42271116
parent 1c6d5f31b1bc9471d52cb6bee2b1e1129a23077f

Remove redundant inline in LoadBalancerServerId::data().
According to go/totw/125:

The inline keyword is not required when the function is defined inside a class or struct, as indicated by C++11 §7.1.2.3:

A function defined within a class definition is an inline function.

PiperOrigin-RevId: 438636469

quic/load_balancer/load_balancer_server_id.h

diff --git a/quic/load_balancer/load_balancer_server_id.h b/quic/load_balancer/load_balancer_server_id.h
index e729437..fe4c5df 100644
--- a/quic/load_balancer/load_balancer_server_id.h
+++ b/quic/load_balancer/load_balancer_server_id.h
@@ -47,7 +47,7 @@
   absl::Span<const uint8_t> data() const {
     return absl::MakeConstSpan(data_.data(), length_);
   }
-  inline uint8_t length() const { return length_; }
+  uint8_t length() const { return length_; }
 
   // Returns the server ID in hex format.
   std::string ToString() const;