Commit Graph

16 Commits

Author SHA1 Message Date
Andy Stewart
7e8d2fb5a6 Use minimal file paths.
Since we start all external commands by cd'ing to the current file's
directory, we can use the file's name instead of its full path.

The exception is `git show :FILE` where we must use a path relative to
the repo root for maximum git compatibility.  It turns out `git
rev-parse` takes a `--show-prefix` argument which makes our relative
path calculation far simpler.

Minimising files paths has these benefits:

- Easier to inspect the generated commands.
- Less opportunity for escaping problems.
- Eliminates possible mismatches betwen absolute paths generated by git
  and absolute paths generated by Vim (crops up with msys/msys2 on
  Windows).

Thanks to @suxpert for helping with this.
2014-02-26 11:52:38 +01:00
Andy Stewart
468474bedf Custom shellescape(arg) to cope better with Windows.
Hopefully :)
2014-01-31 10:38:04 +01:00
Andy Stewart
8541eec8fc Remove unnecessary escaping.
While I can't say I fully understand escaping rules for Windows, I
understand them better now than I did when I copied that code from
another plugin and hoped for the best.

As far as I can tell, this escaping code was written to be used when
manually constructing a system command to execute – which isn't how this
plugin is written.  So this code shouldn't be here.
2014-01-31 10:04:41 +01:00
Andy Stewart
f016924284 Refactor: inline method. 2014-01-30 16:53:14 +01:00
Andy Stewart
5ac69a8bbc Fix typo in utility#escape(). 2014-01-29 16:10:12 +01:00
Andy Stewart
974cd952fc Always use path relative to repo root with git-show.
More recent versions of git support paths relative to current directory.
But for maximum compatibility we need to use paths relative to the
file's repo's root.
2014-01-29 13:47:22 +01:00
Andy Stewart
c7af455c95 Remove unnecessary shell redirection.
While it is aesthetically nicer to discard the unused output from `git
ls-files`, it isn't necessary.  Discarding requires a shell redirection
which exposes us to vim's shell quirks; it's not worth it.
2014-01-29 10:57:20 +01:00
Andy Stewart
5485c37b38 Fix bug in realtime diff of non-current buffer.
This bug meant that a realtime diff of a buffer which wasn't the current
one -- which happens during GitGutterAll() -- diffed the staged version
of the buffer against the current buffer's contents instead of the
buffer's contents.
2014-01-27 14:51:19 +01:00
Andy Stewart
072846e945 Don't place/unplace dummy sign unnecessarily. 2014-01-27 14:15:19 +01:00
Andy Stewart
f2e08dbeb3 Move git-tracking check into diff command.
This avoids shelling out twice per buffer: once to check whether git
knows about the file and once to perform the diff.  Now we simply do
both in one external call.

Profiling showed external calls to git taking ~20ms.  This doesn't seem
too bad but it adds up.
2014-01-27 10:48:47 +01:00
Andy Stewart
e750b5fa21 Eliminate unnecessary string concatenation. 2014-01-11 17:24:11 +01:00
Andy Stewart
7ce6309a01 Don't call getbufvar() with a default value.
Support for a default value in getbufvar() was introduced some time
after Vim 7.3.547 which is the current stable version in some package
repositories.
2014-01-09 10:05:49 +01:00
Andy Stewart
03ca0f8d93 Extract utility functions. 2014-01-08 09:55:54 +01:00
Andy Stewart
b996dd3b53 Remove unnecessary precondition. 2014-01-06 17:11:19 +01:00
Andy Stewart
88623e3fec Speed up snake-to-camel-case conversion. 2014-01-06 11:31:30 +01:00
Andy Stewart
702d27a84f Autoload. 2013-12-17 11:03:52 +01:00