From d2f988027594dae1903005e5ad27f6e3b34cf119 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michael=20K=C3=A4ufl?= Date: Thu, 14 May 2015 12:44:43 +0200 Subject: [PATCH] Traced performance in case of error too This also keeps the usages of 'start' close together. --- lfs/scanner.go | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/lfs/scanner.go b/lfs/scanner.go index 39c4affb..b642f1de 100644 --- a/lfs/scanner.go +++ b/lfs/scanner.go @@ -52,7 +52,11 @@ var z40 = regexp.MustCompile(`\^?0{40}`) // for all Git LFS pointers it finds for that ref. func ScanRefs(refLeft, refRight string) ([]*wrappedPointer, error) { nameMap := make(map[string]string, 0) + start := time.Now() + defer func() { + tracerx.PerformanceSince("scan", start) + }() revs, err := revListShas(refLeft, refRight, refLeft == "", nameMap) if err != nil { @@ -77,8 +81,6 @@ func ScanRefs(refLeft, refRight string) ([]*wrappedPointer, error) { pointers = append(pointers, p) } - tracerx.PerformanceSince("scan", start) - return pointers, nil } @@ -86,7 +88,11 @@ func ScanRefs(refLeft, refRight string) ([]*wrappedPointer, error) { // Git LFS pointers it finds in the index. func ScanIndex() ([]*wrappedPointer, error) { nameMap := make(map[string]*indexFile, 0) + start := time.Now() + defer func() { + tracerx.PerformanceSince("scan-staging", start) + }() revs, err := revListIndex(false, nameMap) if err != nil { @@ -135,8 +141,6 @@ func ScanIndex() ([]*wrappedPointer, error) { pointers = append(pointers, p) } - tracerx.PerformanceSince("scan-staging", start) - return pointers, nil }