lunny/git-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Reduce nesting

Browse Source
This commit is contained in:
Steve Streeting 2016-07-15 10:34:27 +01:00
parent d063337d2f
commit 20b4d8a43d
1 changed files with 24 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
transfer/custom.go
View File

@ -348,32 +348,33 @@ func newCustomAdapter(name string, dir Direction, path, args string, concurrent
func ConfigureCustomAdapters() {
pathRegex := regexp.MustCompile(`lfs.customtransfer.([^.]+).path`)
for k, v := range config.Config.AllGitConfig() {
if match := pathRegex.FindStringSubmatch(k); match != nil {
name := match[1]
path := v
// retrieve other values
args, _ := config.Config.GitConfig(fmt.Sprintf("lfs.customtransfer.%s.args", name))
concurrent := config.Config.GitConfigBool(fmt.Sprintf("lfs.customtransfer.%s.concurrent", name), true)
direction, _ := config.Config.GitConfig(fmt.Sprintf("lfs.customtransfer.%s.direction", name))
if len(direction) == 0 {
direction = "both"
} else {
direction = strings.ToLower(direction)
}
match := pathRegex.FindStringSubmatch(k)
if match == nil {
continue
}
// Separate closure for each since we need to capture vars above
newfunc := func(name string, dir Direction) TransferAdapter {
return newCustomAdapter(name, dir, path, args, concurrent)
}
name := match[1]
path := v
// retrieve other values
args, _ := config.Config.GitConfig(fmt.Sprintf("lfs.customtransfer.%s.args", name))
concurrent := config.Config.GitConfigBool(fmt.Sprintf("lfs.customtransfer.%s.concurrent", name), true)
direction, _ := config.Config.GitConfig(fmt.Sprintf("lfs.customtransfer.%s.direction", name))
if len(direction) == 0 {
direction = "both"
} else {
direction = strings.ToLower(direction)
}
if direction == "download" || direction == "both" {
RegisterNewTransferAdapterFunc(name, Download, newfunc)
}
if direction == "upload" || direction == "both" {
RegisterNewTransferAdapterFunc(name, Upload, newfunc)
}
// Separate closure for each since we need to capture vars above
newfunc := func(name string, dir Direction) TransferAdapter {
return newCustomAdapter(name, dir, path, args, concurrent)
}
if direction == "download" || direction == "both" {
RegisterNewTransferAdapterFunc(name, Download, newfunc)
}
if direction == "upload" || direction == "both" {
RegisterNewTransferAdapterFunc(name, Upload, newfunc)
}
}
}