mirror of
https://github.com/godotengine/godot
synced 2024-09-05 14:30:30 +00:00
Fix editor profiler script function sort order
The engine internally limits the number of functions reported back (to 16 by default). To this point, it's been sort the profiling info in *ascending* order of time spent, then trimming the list. This meant we may only see the best (fastest) functions, instead of the worst that you probably want when profiling. Now the servers_debugger sort more closely matches the local_debugger one, which worked fine.
This commit is contained in:
parent
17e7f85c06
commit
90a5f23e79
|
@ -350,7 +350,7 @@ void EditorProfiler::_update_frame() {
|
|||
category->set_custom_color(0, _get_color_from_signature(m.categories[i].signature));
|
||||
}
|
||||
|
||||
for (int j = m.categories[i].items.size() - 1; j >= 0; j--) {
|
||||
for (int j = 0; j < m.categories[i].items.size(); j++) {
|
||||
const Metric::Category::Item &it = m.categories[i].items[j];
|
||||
|
||||
if (it.internal == it.total && !display_internal_profiles->is_pressed() && m.categories[i].name == "Script Functions") {
|
||||
|
|
|
@ -198,7 +198,7 @@ class ServersDebugger::ScriptsProfiler : public EngineProfiler {
|
|||
typedef ServersDebugger::ScriptFunctionInfo FunctionInfo;
|
||||
struct ProfileInfoSort {
|
||||
bool operator()(ScriptLanguage::ProfilingInfo *A, ScriptLanguage::ProfilingInfo *B) const {
|
||||
return A->total_time < B->total_time;
|
||||
return A->total_time > B->total_time;
|
||||
}
|
||||
};
|
||||
Vector<ScriptLanguage::ProfilingInfo> info;
|
||||
|
|
Loading…
Reference in a new issue