Use 'std::to_string()' instead of 'stringify()'

3rdparty · Feb 20, 2022 · ddad59c · ddad59c
1 parent eded1da
commit ddad59c
Show file tree

Hide file tree

Showing 11 changed files with 11 additions and 25 deletions.
diff --git a/test/cancelled-by-client.cc b/test/cancelled-by-client.cc
@@ -56,7 +56,7 @@ TEST_F(EventualsGrpcTest, CancelledByClient) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/cancelled-by-server.cc b/test/cancelled-by-server.cc
@@ -57,7 +57,7 @@ TEST_F(EventualsGrpcTest, CancelledByServer) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/deadline.cc b/test/deadline.cc
@@ -59,7 +59,7 @@ TEST_F(EventualsGrpcTest, Deadline) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/greeter-server.cc b/test/greeter-server.cc
@@ -60,7 +60,7 @@ TEST_F(EventualsGrpcTest, Greeter) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/multiple-hosts.cc b/test/multiple-hosts.cc
@@ -69,7 +69,7 @@ TEST_F(EventualsGrpcTest, MultipleHosts) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/server-death-test.cc b/test/server-death-test.cc
@@ -83,7 +83,7 @@ TEST_F(EventualsGrpcTest, ServerDeathTest) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/server-unavailable.cc b/test/server-unavailable.cc
@@ -23,7 +23,7 @@ TEST_F(EventualsGrpcTest, ServerUnavailable) {
   // NOTE: we use 'getpid()' to create a _unique_ UNIX domain socket
   // path that should never have a server listening on for this test.
   Client client(
-      "unix:eventuals-grpc-test-server-unavailable-" + stringify(getpid()),
+      "unix:eventuals-grpc-test-server-unavailable-" + std::to_string(getpid()),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/streaming.cc b/test/streaming.cc
@@ -85,7 +85,7 @@ void test_client_behavior(Handler handler) {
                       std::vector<keyvaluestore::Response> responses;
                       for (size_t i = 0; i < 3; i++) {
                         keyvaluestore::Response response;
-                        response.set_value(stringify(i + 10));
+                        response.set_value(std::to_string(i + 10));
                         responses.push_back(response);
                       }
                       return Iterate(std::move(responses));
@@ -101,7 +101,7 @@ void test_client_behavior(Handler handler) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/test.h b/test/test.h
@@ -35,17 +35,3 @@ class EventualsGrpcTest : public ::testing::Test {
 std::filesystem::path GetRunfilePathFor(const std::filesystem::path& runfile);
 
 ////////////////////////////////////////////////////////////////////////
-
-// TODO(benh): Move to stout-stringify.
-template <typename T>
-std::string stringify(const T& t) {
-  std::ostringstream out;
-  out << t;
-  if (!out.good()) {
-    std::cerr << "Failed to stringify!" << std::endl;
-    abort();
-  }
-  return out.str();
-}
-
-////////////////////////////////////////////////////////////////////////
diff --git a/test/unary.cc b/test/unary.cc
@@ -69,7 +69,7 @@ TEST_F(EventualsGrpcTest, Unary) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());
 

diff --git a/test/unimplemented.cc b/test/unimplemented.cc
@@ -38,7 +38,7 @@ TEST_F(EventualsGrpcTest, Unimplemented) {
   Borrowable<CompletionPool> pool;
 
   Client client(
-      "0.0.0.0:" + stringify(port),
+      "0.0.0.0:" + std::to_string(port),
       grpc::InsecureChannelCredentials(),
       pool.Borrow());