Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(649)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: net/tools/quic/quic_client_bin.cc

Issue 2591143003: Add QuicStrCat. (Closed)
Patch Set: correct quic_client_bin.cc Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« net/quic/platform/impl/quic_str_cat_impl.h ('K') | « net/tools/quic/end_to_end_test.cc ('k') | net/tools/quic/quic_dispatcher_test.cc » ('j') | net/tools/quic/quic_http_response_cache.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_client_bin.cc
diff --git a/net/tools/quic/quic_client_bin.cc b/net/tools/quic/quic_client_bin.cc
index 8c979487c62c298428ec1d9d8235585730ffd930..a6fece5a06f8ad66c58b96fc70751cb0e3fe24e4 100644
--- a/net/tools/quic/quic_client_bin.cc
+++ b/net/tools/quic/quic_client_bin.cc
@@ -245,8 +245,7 @@ int main(int argc, char* argv[]) {
net::QuicIpAddress(net::QuicIpAddressImpl(addresses[0].address()));
}
- string host_port =
- base::StringPrintf("%s:%d", ip_addr.ToString().c_str(), port);
+ string host_port = net::QuicStrCat(ip_addr.ToString(), ":", port);
VLOG(1) << "Resolved " << host << " to " << host_port << endl;
// Build the client, and try to connect.
« net/quic/platform/impl/quic_str_cat_impl.h ('K') | « net/tools/quic/end_to_end_test.cc ('k') | net/tools/quic/quic_dispatcher_test.cc » ('j') | net/tools/quic/quic_http_response_cache.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698