D20264: Add test for  "Auto Reload Document" option
    loh tar 
    noreply at phabricator.kde.org
       
    Sat Apr  6 17:27:59 BST 2019
    
    
  
loh.tar updated this revision to Diff 55576.
loh.tar edited the summary of this revision.
loh.tar added a comment.
  - Enhance the test by checks for proper cursor position
  
  While adding your tiny change, it pointed out, that the test was not correct regarding needed waiting times. Pls compare new/old comments about the delays :-/ Any idea to the magic value?
  
  I noticed..
  
  - that slotDelayedHandleModOnHd is still called/processed. I'm not sure if that's OK
  - KateViewInternal::updateView is very often called and so doc()->delayAutoReload() too. The intend was to block the reload while scrolling around. Perhaps could that blocking somewhere else better placed
CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D20264?vs=55468&id=55576
REVISION DETAIL
  https://phabricator.kde.org/D20264
AFFECTED FILES
  autotests/src/katedocument_test.cpp
  autotests/src/katedocument_test.h
  src/document/katedocument.cpp
To: loh.tar, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, #ktexteditor, gennad, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190406/4c18dc4d/attachment.html>
    
    
More information about the Kde-frameworks-devel
mailing list