apol added a comment.

  I'm not sure what we win but I'm not against the optimization.

INLINE COMMENTS

> view.cpp:370
> +    // Avoid removing the same item from the front and prepending it again
> +    if (!m_history.isEmpty() && m_history.first() == item) {
> +        return;

`m_history.constFirst() == item`

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D20383

To: bruns, #plasma, broulik, apol
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to