diff --git a/check.go b/check.go index 5ca345d..96284d1 100644 --- a/check.go +++ b/check.go @@ -8,7 +8,7 @@ package mtree // keywords) and then doing a Compare(dh, newDh, keywords). func Check(root string, dh *DirectoryHierarchy, keywords []string) ([]InodeDelta, error) { if keywords == nil { - keywords = CollectUsedKeywords(dh) + keywords = dh.UsedKeywords() } newDh, err := Walk(root, nil, keywords) @@ -25,7 +25,7 @@ func Check(root string, dh *DirectoryHierarchy, keywords []string) ([]InodeDelta // equivalent to Compare(dh, tarDH, keywords). func TarCheck(tarDH, dh *DirectoryHierarchy, keywords []string) ([]InodeDelta, error) { if keywords == nil { - keywords = CollectUsedKeywords(dh) + keywords = dh.UsedKeywords() } return Compare(dh, tarDH, keywords) } diff --git a/check_test.go b/check_test.go index 08e9211..5cdf756 100644 --- a/check_test.go +++ b/check_test.go @@ -200,7 +200,7 @@ func TestTarTime(t *testing.T) { t.Fatal(err) } - keywords := CollectUsedKeywords(dh) + keywords := dh.UsedKeywords() // make sure "time" keyword works _, err = Check(dir, dh, keywords) diff --git a/cmd/gomtree/main.go b/cmd/gomtree/main.go index d7b745d..cd81344 100644 --- a/cmd/gomtree/main.go +++ b/cmd/gomtree/main.go @@ -142,7 +142,7 @@ func app() error { // We can't check against more fields than in the specKeywords list, so // currentKeywords can only have a subset of specKeywords. - specKeywords = mtree.CollectUsedKeywords(specDh) + specKeywords = specDh.UsedKeywords() } // -list-used diff --git a/hierarchy.go b/hierarchy.go index b0d6dac..88a007d 100644 --- a/hierarchy.go +++ b/hierarchy.go @@ -26,13 +26,9 @@ func (dh DirectoryHierarchy) WriteTo(w io.Writer) (n int64, err error) { return sum, nil } -// CollectUsedKeywords collects and returns all the keywords used in a +// UsedKeywords collects and returns all the keywords used in a // a DirectoryHierarchy -func CollectUsedKeywords(dh *DirectoryHierarchy) []string { - if dh == nil { - return nil - } - +func (dh DirectoryHierarchy) UsedKeywords() []string { usedkeywords := []string{} for _, e := range dh.Entries { switch e.Type {