cls: Fix order of operations of std::string constructor

Presumably the intent was to create a string with 128 '1's, rather than
a string with 49 '\200's.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>

diff --git a/src/cls/journal/cls_journal_types.cc b/src/cls/journal/cls_journal_types.cc
index 6dd5664888cfc6074bd6185c26ff14ccc010be57..c8757e20a90a42a357c46857e76319a81d03dbcd 100644 (file)
--- a/src/cls/journal/cls_journal_types.cc
+++ b/src/cls/journal/cls_journal_types.cc
@@ -100,7 +100,7 @@ void Client::dump(Formatter *f) const {
 
 void Client::generate_test_instances(std::list<Client *> &o) {
   bufferlist data;
-  data.append(std::string('1', 128));
+  data.append(std::string(128, '1'));
 
   o.push_back(new Client());
   o.push_back(new Client("id", data));
@@ -136,7 +136,7 @@ void Tag::generate_test_instances(std::list<Tag *> &o) {
   o.push_back(new Tag());
 
   bufferlist data;
-  data.append(std::string('1', 128));
+  data.append(std::string(128, '1'));
   o.push_back(new Tag(123, 234, data));
 }