From 1ac6099100ed61a1e49766f190deae8b426a1ea6 Mon Sep 17 00:00:00 2001
From: Runxi Yu
Date: Sun, 22 Mar 2026 22:52:35 +0000
Subject: commitgraph: Tighten docs and use a value-ish Filter return

---
 commitgraph/read/bloom.go | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'commitgraph/read/bloom.go')

diff --git a/commitgraph/read/bloom.go b/commitgraph/read/bloom.go
index c7e3e426..eca88c36 100644
--- a/commitgraph/read/bloom.go
+++ b/commitgraph/read/bloom.go
@@ -38,20 +38,23 @@ func (reader *Reader) BloomVersion() uint8 {
 
 // BloomFilterAt returns one commit's changed-path Bloom filter.
 //
+// The returned filter borrows reader-owned mapped commit-graph data and is
+// only valid until the reader is closed.
+//
 // Returns BloomUnavailableError when this commit graph has no Bloom data.
-func (reader *Reader) BloomFilterAt(pos Position) (*bloom.Filter, error) {
+func (reader *Reader) BloomFilterAt(pos Position) (bloom.Filter, error) {
 	layer, err := reader.layerByPosition(pos)
 	if err != nil {
-		return nil, err
+		return bloom.Filter{}, err
 	}
 
 	if layer.chunkBloomIndex == nil || layer.chunkBloomData == nil || layer.bloomSettings == nil {
-		return nil, &BloomUnavailableError{Pos: pos}
+		return bloom.Filter{}, &BloomUnavailableError{Pos: pos}
 	}
 
 	start, end, err := bloomRange(layer, pos.Index)
 	if err != nil {
-		return nil, err
+		return bloom.Filter{}, err
 	}
 
 	filter := bloom.NewFilter(
-- 
cgit v1.3.1-10-gc9f91