Merge branch 'n/transfers' into next
magit-fetch-current: cosmetics, fix doc-string
magit-remote-rename: prompt cosmetics

If you think I should not have done that then you might want to
get of the computer and train your short time memory instead :-)
magit-diff-highlight-lines: keep foreground of other lines

By default only change the background color of lines that are outside
the region selecting lines in a hunk section.  Faces for added/removed
to use distinct colors for the background as well as the foreground, by
only changing the background to be identical to that of context lines,
added/removed lines outside the region look different from context
lines, and at the same time look different from selected added/removed
lines.

Add the variable `magit-diff-unmarked-lines-keep-foreground' setting
it to nil restores the old look.

Closes #1583.
magit-read-file-from-rev: expect non-nil REVISION

The argument is mandatory, and if it is nil then that
is a bug in the caller and should be fixed there.
magit-read-file-from-rev: cosmetics
magit-revision-files: return all files

Return all files in the revision, not
just those below the current directory.
magit-read-file-{from-rev,hist}: move definitions
magit-find-file-read-args: cosmetics
magit-read-file-trace: remove duplicated definition
magit-log-read-args: move definition
magit-log-read-args: rename arguments
magit-log-popup: where available use --long arguments
magit-log-popup: improve argument descriptions

Improve the descriptions of various switches and options.  Also no
longer mark "verbose-only" arguments as such, we will soon do that
differently.
magit-log-popup: improve action descriptions
magit-log-popup: reorder arguments
magit-log-popup: remove adventurous -L and -S arguments

The arguments `-L' and `-S' should never have been added to the log
popup.  They could be very useful to implement fancy history navigation
but just dropping them into the popup as is, really doesn't cut it.

Also remove `magit-read-file-trace' which was only used for `-L'.
magit-log-popup: remove broken --name-only argument
magit-log-popup: bind lowercase keys to arguments, most actions

For keys that change the values of arguments, give up on only using
keys corresponding to the short form of the respective Git argument.
Instead use all lowercase letters and try to make them consistent
within this popup.

For action keys we cannot do completely without uppercase letters,
we use uppercase for the "verbose" variants.  However use "o"/"O"
instead of "b"/"B", for consistency with other popups where we also
use the former for the "act on something other than the default"
action variants.
magit-log-popup: remove most expert arguments

In the past I have been criticized for both, (a) adding useless
confusing arguments, and (b) refusing to add essential arguments.

Different users have different needs and I won't be able to make
everyone happy.  Here I have chosen to remove lesser used arguments
which would confuse the non-expert.  The expert is welcome to add
these arguments back, I have invested quite some time making that
possible - use the functions `magit-define-popup-switch' and
`magit-define-popup-option'.