From f84e718476687a7c520241fb739e7a201ca143d1 Mon Sep 17 00:00:00 2001
From: Emil Ernerfeldt <emil.ernerfeldt@gmail.com>
Date: Wed, 18 Dec 2024 15:20:47 +0100
Subject: [PATCH] Give proper attribution for `ZipValidity`

---
 crates/store/re_types_core/src/arrow_zip_validity.rs | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/crates/store/re_types_core/src/arrow_zip_validity.rs b/crates/store/re_types_core/src/arrow_zip_validity.rs
index 0b67eab8a665..20194063b924 100644
--- a/crates/store/re_types_core/src/arrow_zip_validity.rs
+++ b/crates/store/re_types_core/src/arrow_zip_validity.rs
@@ -1,5 +1,10 @@
-//! Code adapted from <https://github.com/jorgecarleitao/arrow2/blob/main/src/bitmap/utils/zip_validity.rs>
-// TODO: rename and maybe rewrite
+//! Easily iterate over arrow values that may contain nulls.
+//!
+//! Code adapted from <https://github.com/jorgecarleitao/arrow2/blob/65c0ac4182748ec39b0a158ec5ebde829aae12b6/src/bitmap/utils/zip_validity.rs>
+//! Originally written by [Jorge Leitao](https://github.com/jorgecarleitao),
+//! under Apache License 2.0 (see `LICENSE-APACHE` file).
+//!
+//! Adapted by Rerun to work with `arrow` crate.
 
 use arrow::{buffer::NullBuffer, util::bit_iterator::BitIterator};
 
@@ -95,6 +100,7 @@ where
 {
     /// There are no null values
     Required(I),
+
     /// There are null values
     Optional(ZipValidityIter<T, I, V>),
 }