D7595: Skip reload when another reload is already in progress
    Henrik Fehlauer 
    noreply at phabricator.kde.org
       
    Wed Aug 30 06:26:28 UTC 2017
    
    
  
rkflx added a comment.
  I guess someone else should judge which is easier to read considering the full picture (I got confused by your `areWeReloading` naming, actually). I admit someone not familiar with `std::mutex` semantics would need to read the comment, but understanding a custom RAII wrapper is even harder.
  
  Could you suggest another reviewer?
REPOSITORY
  R223 Okular
REVISION DETAIL
  https://phabricator.kde.org/D7595
To: rkflx, #okular
Cc: aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20170830/4a58b35c/attachment.html>
    
    
More information about the Okular-devel
mailing list