Document that wrap_optimal_fit panics

mgeisler · Jan 4, 2021 · 2eb83f3 · 2eb83f3
1 parent 59a20b3
commit 2eb83f3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/fuzz/fuzz_targets/wrap_optimal_fit.rs b/fuzz/fuzz_targets/wrap_optimal_fit.rs
@@ -23,6 +23,8 @@ fuzz_target!(|input: (Vec<BoxGluePenalty>, u64)| {
             .and_then(|sum| sum.checked_add(f.penalty_width()))
     });
     if total_width.is_none() {
+        // The total width overflows usize — this is not supported so
+        // return here to avoid a crash.
         return;
     }
 

diff --git a/src/core/optimal_fit.rs b/src/core/optimal_fit.rs
@@ -151,6 +151,11 @@ const HYPHEN_PENALTY: usize = 25;
 /// code by David
 /// Eppstein](https://github.com/jfinkels/PADS/blob/master/pads/wrap.py).
 ///
+/// # Panics
+///
+/// The total width of all fragments must fit inside an `usize`
+/// (including the whitespace and penalty widths).
+///
 /// **Note:** Only available when the `smawk` Cargo feature is
 /// enabled.
 pub fn wrap_optimal_fit<'a, T: Fragment, F: Fn(usize) -> usize>(