D9809: work around bug in kLineEdit

[Dan Weatherill]

dweatherill added a comment.


  In https://phabricator.kde.org/D9809#189484, @mwolff wrote:
  
  > So just remove the assert - instead return early when indexes.isEmpty(). That way we don't need to mess with the signal blocker and the code behaves as intended. And you remove code instead of adding more.
  
  
  I agree that fixes the crash, in fact that was the first fix I tried.
  
  However, compilerEdited is still being spuriously called, and the underlying cause is the kLineEdit bug. 
  When that bug gets fixed, we could do a version check on KF5 libs and #ifdef out this fix. In that case, I think it's clearer to block the signals... but I am happy to change it if that is the preferred solution

REPOSITORY
  R32 KDevelop

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

To: dweatherill, #kdevelop, apol, mwolff
Cc: mwolff, anthonyfieroni, kdevelop-devel, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180111/483b37c6/attachment-0001.html>