Merge lp:~pauljnixon/bzr/context_in_diffs into lp:bzr
Proposed by
Paul Nixon
Status: | Superseded |
---|---|
Proposed branch: | lp:~pauljnixon/bzr/context_in_diffs |
Merge into: | lp:bzr |
Diff against target: |
226 lines (+91/-12) 4 files modified
bzrlib/builtins.py (+7/-2) bzrlib/diff.py (+17/-10) bzrlib/tests/test_diff.py (+63/-0) doc/en/release-notes/bzr-2.6.txt (+4/-0) |
To merge this branch: | bzr merge lp:~pauljnixon/bzr/context_in_diffs |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
bzr-core | Pending | ||
Review via email: mp+113793@code.launchpad.net |
This proposal has been superseded by a proposal from 2012-07-19.
Description of the change
Patiencediff (used by default for "bzr diff" command) includes the ability to control how many lines of context to output for each changed section. I made this accessible as an option to bzr diff, as "--context" so an example would be "bzr diff -r-2 --context 6"
I have also made this accessible to Loggerhead which I will propose for merging if this goes in.
This is the first time I have proposed anything for merging, if I'm doing something wrong just let me know and I'll fix it.
To post a comment you must log in.
Hi Paul,
The changes don't look unreasonable. At a first glance, two notes: it would be nice to have a constant for the default number of context lines. Can you please add some basic tests for the new option?
Cheers,
Jelmer
--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.
Paul <email address hidden> wrote:
Paul has proposed merging lp:~pauljnixon/bzr/context_in_diffs into lp:bzr.
Requested reviews:
bzr-core (bzr-core)
For more details, see: /code.launchpad .net/~pauljnixo n/bzr/context_ in_diffs/ +merge/ 113793
https:/
Patiencediff (used by default for "bzr diff" command) includes the ability to control how many lines of context to output for each changed section. I made this accessible as an option to bzr diff, as "--context" so an example would be "bzr diff -r-2 --context 6"
I have also made this accessible to Loggerhead which I will propose for merging if this goes in.
This is the first time I have proposed anything for merging, if I'm doing something wrong just let me know and I'll fix it. /code.launchpad .net/~pauljnixo n/bzr/context_ in_diffs/ +merge/ 113793
--
https:/
Your team Bazaar Codereview Subscribers is subscribed to branch lp:bzr.