From 9859533ab461c6d509337d8b9013a7ba62dd78ae Mon Sep 17 00:00:00 2001 From: Lioncash Date: Sun, 26 Mar 2017 22:17:37 -0400 Subject: VertexLoaderBase: Return debug strings by value An out parameter for this sort of thing is a C++03 hold-over. This also renames AppendToString to ToString. --- Source/Core/VideoCommon/VertexLoaderManager.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'Source/Core/VideoCommon/VertexLoaderManager.cpp') diff --git a/Source/Core/VideoCommon/VertexLoaderManager.cpp b/Source/Core/VideoCommon/VertexLoaderManager.cpp index c7a2f46bab..f0d8decb3d 100644 --- a/Source/Core/VideoCommon/VertexLoaderManager.cpp +++ b/Source/Core/VideoCommon/VertexLoaderManager.cpp @@ -106,11 +106,11 @@ void AppendListToString(std::string* dest) size_t total_size = 0; for (const auto& map_entry : s_vertex_loader_map) { - entry e; - map_entry.second->AppendToString(&e.text); - e.num_verts = map_entry.second->m_numLoadedVertices; - entries.push_back(e); + entry e = {map_entry.second->ToString(), + static_cast(map_entry.second->m_numLoadedVertices)}; + total_size += e.text.size() + 1; + entries.push_back(std::move(e)); } sort(entries.begin(), entries.end()); dest->reserve(dest->size() + total_size); -- cgit v1.2.3 From 9ebd84e54afecee306b4bbc97fa15604def37310 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Sun, 26 Mar 2017 22:52:10 -0400 Subject: VertexLoaderManager: Return debug strings by value This also renames AppendListToString to VertexLoadersToString. --- Source/Core/VideoCommon/VertexLoaderManager.cpp | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) (limited to 'Source/Core/VideoCommon/VertexLoaderManager.cpp') diff --git a/Source/Core/VideoCommon/VertexLoaderManager.cpp b/Source/Core/VideoCommon/VertexLoaderManager.cpp index f0d8decb3d..463094e66b 100644 --- a/Source/Core/VideoCommon/VertexLoaderManager.cpp +++ b/Source/Core/VideoCommon/VertexLoaderManager.cpp @@ -98,7 +98,7 @@ struct entry }; } -void AppendListToString(std::string* dest) +std::string VertexLoadersToString() { std::lock_guard lk(s_vertex_loader_map_lock); std::vector entries; @@ -112,13 +112,18 @@ void AppendListToString(std::string* dest) total_size += e.text.size() + 1; entries.push_back(std::move(e)); } + sort(entries.begin(), entries.end()); - dest->reserve(dest->size() + total_size); + + std::string dest; + dest.reserve(total_size); for (const entry& entry : entries) { - *dest += entry.text; - *dest += '\n'; + dest += entry.text; + dest += '\n'; } + + return dest; } void MarkAllDirty() -- cgit v1.2.3