<table><tr><td style="">rjvbb edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-yvadpqbs5pgx7ko/" rel="noreferrer">(Show Details)</a><br />rjvbb edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-yxoddxuwt652h43/" rel="noreferrer">(Show Details)</a><br />rjvbb set the repository for this revision to R33 KDevPlatform.
</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/D4981" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>One example is the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">(admittedly) rare </span>kind of patch that can be applied multiple times,<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> repeating the programmed difference each time. That actually happens quite often to me.<br />
<br />
Another example is uploading a so-called "raw diff" (changes not committed locally) to Phabricator, using the Purpose/Phabricator plugin or otherwise. Such patches would normally lack context beyond what is contained in the patch itself.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">repeat</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Increas</span>ing the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">programmed difference each time</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">number of lines makes the review process easier</span>.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Another example is uploading a so-called "raw diff" (changes not committed locally) to Phabricator, using the Purpose/Phabricator plugin or otherwisee. Such patches would normally lack context beyond what is contained in the patch itself</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This patch introduces a subtle extension to the patchreview UI that allows to select a custom number of context lines: it adds a context menu to the Update button that offers a selection of context lines plus the special "whole file" value to include the full file as context</span>. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Increasing the number of lines could make the review process easier.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Once a number has been selected it will be be used each time the Update button is clicked or the diff is updated for other reasons,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">(NB: integration with Phab should still be improved to include support of Phab's preferred way of submitting "locally committed patches".)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">for the lifetime of the current patch review process.</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">This patch introduces a simple spinbox to the patchreview UI that allows to select a custom number of context lines. The spinbox has a special value "max" (= 0) which is translated internally to `INT_MAX`. The initial diff of this revision was created that way, and contains thus entire copies of the changed fil</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Uploading raw diffs generated with the "whole file" option to Phabricator provides reviewers with the same reviewing convenience as uploading committed differenc</span>es.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Seems to work</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Works</span> as intended as far as the custom context feature has been implemented in the various VCS plugins.<br />
<br />
It would be nice if the patchfile window reloaded automatically whenever the file changes<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> (on all platforms; works on Linux, not on Mac)</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4981" rel="noreferrer">https://phabricator.kde.org/D4981</a></div></div><br /><div><strong>To: </strong>rjvbb, KDevelop, kfunk<br /><strong>Cc: </strong>kfunk, apol, kdevelop-devel, KDevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>