Le 14/06/2016 17:01, Jean-Marc Lasgouttes a écrit :
Le 12/06/2016 à 20:23, Guillaume Munch a écrit :
commit 89175ee0f10f75311e2a746dbd450fb23d6c2845
Author: Guillaume Munch <[email protected]>
Date:   Sat Jun 11 09:14:38 2016 +0100

    Add perf comment

diff --git a/src/KeyMap.cpp b/src/KeyMap.cpp
index 7628d0d..32f9926 100644
--- a/src/KeyMap.cpp
+++ b/src/KeyMap.cpp
@@ -106,6 +106,8 @@ void KeyMap::bind(KeySequence * seq, FuncRequest
const & func, unsigned int r)
     KeyModifier const mod2 = seq->modifiers[r].second;

     // check if key is already there
+    // FIXME perf: Profiling shows that this is responsible of 99% of
the
+    // cost of GuiPrefs::applyView()
     Table::iterator end = table.end();

Interesting. Is it only the loop in itself that is costly, or some of
the functions it calls?


The combined loop and recursion, I believe.


Reply via email to