From 3447303ce76caa3a22e9f510fa00e73f5143267e Mon Sep 17 00:00:00 2001 From: Jeremy Stribling Date: Wed, 1 Nov 2017 12:01:36 -0700 Subject: filesystem: todo comment about "all" param Issue: KBFS-2517 --- storage/filesystem/internal/dotgit/dotgit.go | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'storage/filesystem/internal') diff --git a/storage/filesystem/internal/dotgit/dotgit.go b/storage/filesystem/internal/dotgit/dotgit.go index 6d56318..330f9d5 100644 --- a/storage/filesystem/internal/dotgit/dotgit.go +++ b/storage/filesystem/internal/dotgit/dotgit.go @@ -597,6 +597,10 @@ func (d *DotGit) CountLooseRefs() (int, error) { // ref update could also lock packed-refs, so only one lock is // required during ref-packing. But that would worsen performance in // the common case. +// +// TODO: add an "all" boolean like the `git pack-refs --all` flag. +// When `all` is false, it would only pack refs that have already been +// packed, plus all tags. func (d *DotGit) PackRefs() (err error) { // Lock packed-refs, and create it if it doesn't exist yet. f, err := d.fs.Open(packedRefsPath) -- cgit