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

Unified Diff: chrome/browser/net/dns_host_info.cc

Issue 339059: Add compiler-specific "examine printf format" attributes to printfs. (Closed)
Patch Set: cleanups Created 11 years, 1 month 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
Index: chrome/browser/net/dns_host_info.cc
diff --git a/chrome/browser/net/dns_host_info.cc b/chrome/browser/net/dns_host_info.cc
index dbfde3bfc1c5403200765c5668f42fee6ae9f5d5..63eae347f47ada7ce62f2d0d1664190bebf35eb0 100644
--- a/chrome/browser/net/dns_host_info.cc
+++ b/chrome/browser/net/dns_host_info.cc
@@ -2,8 +2,6 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-// See header file for description of class
-
#include "chrome/browser/net/dns_host_info.h"
#include <math.h>
@@ -11,6 +9,7 @@
#include <algorithm>
#include <string>
+#include "base/format_macros.h"
#include "base/histogram.h"
#include "base/logging.h"
#include "base/string_util.h"
@@ -326,7 +325,7 @@ void DnsHostInfo::GetHtmlTable(const DnsInfoTable host_infos,
if (0 == host_infos.size())
return;
output->append(description);
- StringAppendF(output, "%d %s", host_infos.size(),
+ StringAppendF(output, "%" PRIuS " %s", host_infos.size(),
(1 == host_infos.size()) ? "hostname" : "hostnames");
if (brief) {

Powered by Google App Engine
This is Rietveld 408576698