ASSERT_EQ(-ENOENT, client::get_client(ioctx, oid, "id", &client));
 
   bufferlist data;
-  data.append(std::string('1', 128));
+  data.append(std::string(128, '1'));
   ASSERT_EQ(0, client::client_register(ioctx, oid, "id1", data));
 
   ASSERT_EQ(0, client::get_client(ioctx, oid, "id1", &client));
   ASSERT_EQ(0, client::client_register(ioctx, oid, "id1", bufferlist()));
 
   bufferlist data;
-  data.append(std::string('1', 128));
+  data.append(std::string(128, '1'));
   ASSERT_EQ(0, client::client_update_data(ioctx, oid, "id1", data));
 
   Client client;
   ASSERT_EQ(0, client::client_register(ioctx, oid, "id1", bufferlist()));
 
   bufferlist data;
-  data.append(std::string('1', 128));
+  data.append(std::string(128, '1'));
   ASSERT_EQ(0, client::client_update_state(ioctx, oid, "id1",
                                            CLIENT_STATE_DISCONNECTED));