Skip to content

Commit

Permalink
Fix missing const-reference on parameter.
Browse files Browse the repository at this point in the history
This issue was found by a linter.

R=nick@chromium.org

BUG=

Review URL: https://chromiumcodereview.appspot.com/23286004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218887 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
etienneb@chromium.org committed Aug 22, 2013
1 parent ecd434a commit f7c5c88
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions sync/syncable/syncable_util.cc
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,8 @@ std::string GenerateSyncableHash(
}

std::string GenerateSyncableBookmarkHash(
const std::string originator_cache_guid,
const std::string originator_client_item_id) {
const std::string& originator_cache_guid,
const std::string& originator_client_item_id) {
return syncable::GenerateSyncableHash(
BOOKMARKS, originator_cache_guid + originator_client_item_id);
}
Expand Down
4 changes: 2 additions & 2 deletions sync/syncable/syncable_util.h
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ SYNC_EXPORT_PRIVATE std::string GenerateSyncableHash(
// than one place, so we define the algorithm here to make sure the
// implementation is consistent.
SYNC_EXPORT_PRIVATE std::string GenerateSyncableBookmarkHash(
const std::string originator_cache_guid,
const std::string originator_client_item_id);
const std::string& originator_cache_guid,
const std::string& originator_client_item_id);

} // namespace syncable
} // namespace syncer
Expand Down

0 comments on commit f7c5c88

Please sign in to comment.