D22182: Remove anchorClicked lock which causes deadlock

Michael Swan noreply at phabricator.kde.org
Mon Jul 1 02:55:08 BST 2019


mswan created this revision.
Herald added a project: KDevelop.
Herald added a subscriber: kdevelop-devel.
mswan requested review of this revision.

REVISION SUMMARY
  This change fixes a deadlock discussed in bug #358787. This lock was introduced to solve bug #386901.
  I looked through all of the paths to the failed assertion discussed in that latter ticket and it appears that all but
  the invocation in PotentialBuddyCollector::accept is verified to have the reader lock held prior to invocation. There
  were two paths which needed a lock added given my change, so this change set should not cause any regression on
  bug #386901.

REPOSITORY
  R32 KDevelop

BRANCH
  fix-anchorClicked

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

AFFECTED FILES
  kdevplatform/language/duchain/navigation/abstractnavigationwidget.cpp

To: mswan
Cc: kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190701/8a2a5b59/attachment.html>


More information about the KDevelop-devel mailing list