Skip to content

Commit

Permalink
doc: clarify meaning of SetMatches::len
Browse files Browse the repository at this point in the history
It is really unfortunate, but SetMatches::len and
SetMatcher::iter().count() do not correspond go the same thing. It's not
clear why I even added the SetMatches::len method in the first place,
but it always returns the number of regexes in the set, and not the
number of regexes that matched.

We can't change the name (or remove the method) obviously, but we do add
a warning to the docs.

Fixes #625
  • Loading branch information
BurntSushi committed Mar 15, 2023
1 parent f0811ed commit a70ea30
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/re_set.rs
Original file line number Diff line number Diff line change
Expand Up @@ -321,6 +321,11 @@ impl SetMatches {
}

/// The total number of regexes in the set that created these matches.
///
/// **WARNING:** This always returns the same value as [`RegexSet::len`].
/// In particular, it does *not* return the number of elements yielded by
/// [`SetMatches::iter`]. The only way to determine the total number of
/// matched regexes is to iterate over them.
pub fn len(&self) -> usize {
self.matches.len()
}
Expand Down

0 comments on commit a70ea30

Please sign in to comment.