Skip to content

Commit

Permalink
cleaner comment explaining string lengths
Browse files Browse the repository at this point in the history
  • Loading branch information
danielsn committed May 21, 2024
1 parent 11a7539 commit edc1524
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions profiling/src/collections/string_table/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -238,8 +238,9 @@ mod tests {
/// as well as functional correctness (the table should behave like an
/// ordered set).
/// Limitations:
/// - The length of strings appears to be limited: `assert!(string.len() < 50);` failed, but
/// `100` passed (in 10s)
/// - The crate used here to generate Strings internally has a default range for the length of a string, (0..=64)
/// We should experiment with longer strings to see what happens.
/// https://github.com/camshaft/bolero/blob/f401669697ffcbe7f34cbfd09fd57b93d5df734c/lib/bolero-generator/src/alloc/mod.rs#L17
/// - Since iterating is destructive, can only check the string values once.
/// `cargo +nightly bolero test collections::string_table::tests::fuzz_string_table -T 1min`
#[test]
Expand Down

0 comments on commit edc1524

Please sign in to comment.