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

Reply via email to