Toggleable --full-diff option about tig HOT 7 CLOSED

I would prefer a future-proof solution, which can also be applied to possible new options. Something like the following perhaps:

bind generic 2 :set diff-options = "--full-diff"; :reload

Which would require

• A new option called 'diff-options' similar to the existing 'blame-options'.
• The possibility to bind 'tig scriptlets' to keybindings.
• Improvements to the code reading and executing 'tig scripts'.

from tig.

I think adding user defined diff options may break some features in the diff view (e.g. --names-only or --color-words`). Am I mistaken?

from tig.

It's certainly possible for arbitrary diff options to break tig's parsing of the git-diff output. The same is probably true, however, of the existing blame-options.

from tig.

@jonas: Should diff-options replace the TIG_DIFF_OPTS environment variable, or coexist peacefully? I'd opt to replace the env var, but there may be particular value in the per-invocation uses of it.

from tig.

I don't see a reason to remove it, but we could start a deprecation process to avoid any surprises.

from tig.

Nod. I already have code that coexists with TIG_DIFF_OPTS, so I'm fine leaving it for now. I'll document the precedence of each method.

from tig.

Great

On Wed, Jan 23, 2013 at 11:00 PM, Thomas Sibley [email protected]:

Nod. I already have code that coexists with TIG_DIFF_OPTS, so I'm fine
leaving it for now. I'll document the precedence of each method.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/69#issuecomment-12636144.

Jonas Fonseca

from tig.