diff options
author | 2015-06-01 13:52:23 -0400 | |
---|---|---|
committer | 2015-06-01 13:52:23 -0400 | |
commit | 39c13f91004b41166b786785083e78b34df0c48f (patch) | |
tree | bf5ea0acf27c9075a63d3b23b970c3ea44ecee64 /Annex/Branch.hs | |
parent | 3a9ca30fa8740a57ea477243498339f9738102d6 (diff) |
remove Params constructor from Utility.SafeCommand
This removes a bit of complexity, and should make things faster
(avoids tokenizing Params string), and probably involve less garbage
collection.
In a few places, it was useful to use Params to avoid needing a list,
but that is easily avoided.
Problems noticed while doing this conversion:
* Some uses of Params "oneword" which was entirely unnecessary
overhead.
* A few places that built up a list of parameters with ++
and then used Params to split it!
Test suite passes.
Diffstat (limited to 'Annex/Branch.hs')
-rw-r--r-- | Annex/Branch.hs | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Annex/Branch.hs b/Annex/Branch.hs index 4bd94bddb..1a57e2342 100644 --- a/Annex/Branch.hs +++ b/Annex/Branch.hs @@ -315,7 +315,10 @@ files = do - and without updating the branch. -} branchFiles :: Annex [FilePath] branchFiles = withIndex $ inRepo $ Git.Command.pipeNullSplitZombie - [ Params "ls-tree --name-only -r -z" + [ Param "ls-tree" + , Param "--name-only" + , Param "-r" + , Param "-z" , Param $ fromRef fullname ] |