Add a space between string literals and PRIuS

chrome/browser/sync/glue/tab_node_pool.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sync/glue/tab_node_pool.h"
 
 #include "base/format_macros.h"
 #include "base/logging.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
@@ skipping 14 matching lines @@
     : tab_pool_fp_(-1),
       sync_service_(sync_service) {
 }
 
 TabNodePool::~TabNodePool() {}
 
 // Static
 std::string TabNodePool::TabIdToTag(
     const std::string machine_tag,
     size_t tab_node_id) {
-  return base::StringPrintf("%s %"PRIuS"", machine_tag.c_str(), tab_node_id);
+  return base::StringPrintf("%s %" PRIuS "", machine_tag.c_str(), tab_node_id);

[Nico, 2013/04/05 17:05:37]

(I think you only need the space in front of PRIus, not after it)

[stuartmorgan, 2013/04/05 18:48:08]

Yeah, but I thought that looked really strange since the PRIuS is logically
linked to the %, not the part after, so this seemed clearer. I can change it if
you prefer though.

 }
 
 void TabNodePool::AddTabNode(int64 sync_id) {
   tab_syncid_pool_.resize(tab_syncid_pool_.size() + 1);
   tab_syncid_pool_[static_cast<size_t>(++tab_pool_fp_)] = sync_id;
 }
 
 int64 TabNodePool::GetFreeTabNode() {
   DCHECK_GT(machine_tag_.length(), 0U);
   if (tab_pool_fp_ == -1) {
@@ skipping 36 matching lines @@
   }
 }
 
 void TabNodePool::FreeTabNode(int64 sync_id) {
   // Pool size should always match # of free tab nodes.
   DCHECK_LT(tab_pool_fp_, static_cast<int64>(tab_syncid_pool_.size()));
   tab_syncid_pool_[static_cast<size_t>(++tab_pool_fp_)] = sync_id;
 }
 
 }  // namespace browser_sync