minimize calls to len()

Author: adriangb

src/input/iterator.rs

@@ -63,6 +63,11 @@ impl<'data> IterableValidationChecks<'data> {
         };
         self.input_length += 1;
         self.check_max_length(self.input_length, self.length_constraints.max_input_length, input)?;
+        self.check_max_length(
+            self.output_length + self.errors.len(),
+            self.length_constraints.max_length,
+            input,
+        )?;
         res
     }
 

src/validators/dict.rs

@@ -118,8 +118,8 @@ where
         );
         if let Some((key, value)) = checks.filter_validation_result(result, input)? {
             output.set_item(key, value)?;
+            checks.check_output_length(output.len(), input)?;
         }
-        checks.check_output_length(output.len(), input)?;
     }
     checks.finish(input)?;
     Ok(())

src/validators/list.rs

@@ -192,8 +192,8 @@ where
             .map_err(|e| e.with_outer_location(index.into()));
         if let Some(value) = checks.filter_validation_result(result, input)? {
             output.push(value);
+            checks.check_output_length(output.len(), input)?;
         }
-        checks.check_output_length(output.len(), input)?;
     }
     checks.finish(input)?;
     Ok(())

src/validators/sets.rs

@@ -306,8 +306,8 @@ where
             .map_err(|e| e.with_outer_location(index.into()));
         if let Some(value) = checks.filter_validation_result(result, input)? {
             output.add(value)?;
+            checks.check_output_length(output.len(), input)?;
         }
-        checks.check_output_length(output.len(), input)?;
     }
     checks.finish(input)?;
     Ok(())

src/validators/tuple.rs

@@ -71,8 +71,8 @@ where
             .map_err(|e| e.with_outer_location(index.into()));
         if let Some(value) = checks.filter_validation_result(result, input)? {
             output.push(value);
+            checks.check_output_length(output.len(), input)?;
         }
-        checks.check_output_length(output.len(), input)?;
     }
     checks.finish(input)?;
     Ok(())
@@ -241,6 +241,7 @@ where
                     .map_err(|e| e.with_outer_location(index.into()));
                 if let Some(value) = checks.filter_validation_result(result, input)? {
                     output.push(value);
+                    checks.check_output_length(output.len(), input)?;
                 }
             }
             None => {
@@ -252,6 +253,7 @@ where
                             .map_err(|e| e.with_outer_location(index.into()));
                         if let Some(value) = checks.filter_validation_result(result, input)? {
                             output.push(value);
+                            checks.check_output_length(output.len(), input)?;
                         }
                     }
                     None => {
@@ -267,7 +269,6 @@ where
                 }
             }
         }
-        checks.check_output_length(output.len(), input)?;
     }
     for (idx, validator) in items_validators.iter().enumerate().skip(output.len()) {
         let default = validator.default_value(py, Some(output.len()), extra, definitions, recursion_guard)?;