+ # if the user requested function names in the diff
+ # we use the external diff
My main issue with this kind of change is that actually using an external diff rather than the bzr internal one *can* be a big behaviour change. Some people don't have an external diff installed at all, and encoding handling is completely different.
If it's happening in non-obvious circumstances, we risk confusing users over when diff works the way they expect and when it breaks. Implementing '-F' in the bzr internal differ is way more work, but would also do what you want here I think.
+ # if the user requested function names in the diff
+ # we use the external diff
My main issue with this kind of change is that actually using an external diff rather than the bzr internal one *can* be a big behaviour change. Some people don't have an external diff installed at all, and encoding handling is completely different.
If it's happening in non-obvious circumstances, we risk confusing users over when diff works the way they expect and when it breaks. Implementing '-F' in the bzr internal differ is way more work, but would also do what you want here I think.