Fix the cursorUp -> bottom failure: the problem is that switchList does *not* necessarily switch list, so we should not toggle the m_inCompletionList unconditionally. This patch only updates m_inCompletionList if switching was successful, and having m_inCompletionList be incorrect was what caused the problem with bottom().
author Simon St James
()
committer Simon St James
()
commit 782928b4f6bbbdd4436599c8a13db7fa45990893
tree acc765bb63d62e16a8b7305ddc60535825532b08
parent 5196be1ba2d6474dd101166e73da83094b5ea665
Fix the cursorUp -> bottom failure: the problem is that switchList does *not* necessarily switch list, so we should not toggle the m_inCompletionList unconditionally. This patch only updates m_inCompletionList if switching was successful, and having m_inCompletionList be incorrect was what caused the problem with bottom().
part/completion/katecompletionwidget.cpp
part/tests/completion_test.cpp
part/tests/completion_test.h