<table><tr><td style="">rkflx added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7671" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>For completeness, for a file watch on "x" (not sure whether you may have confused file and dir watches as well as file and dir Okular code pathes in your reply), we get:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">touch x</tt> (no x yet) → "created" signal</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">touch x</tt> (again) → "dirty" signal</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">touch ../y && mv ../y x</tt> → "deleted" signal and "created" signal</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">rm x</tt> → "deleted" signal</li>
</ul>

<p>To me this seems logical and consistent, only the documentation regarding "move" is lacking. Of course we could argue that <tt style="background: #ebebeb; font-size: 13px;">touch</tt> (no x yet), <tt style="background: #ebebeb; font-size: 13px;">mv</tt> and <tt style="background: #ebebeb; font-size: 13px;">rm</tt> should also send an additional "dirty" signal. Above I said "it should not", because it breaks backwards compatibility with existing KDirWatch users (they get signals they may have wanted to avoid, or they have a separate code path for "create") and there are already more specialized signals. In addition, only adding to the documentation is less risky than invasive changes to KDirWatch's internals (it has to work for all backends and on all platforms, after all).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7671" rel="noreferrer">https://phabricator.kde.org/D7671</a></div></div><br /><div><strong>To: </strong>progwolff, aacid<br /><strong>Cc: </strong>sander, rkflx, Okular, aacid<br /></div>