Skip to content

Commit c05abde

Browse files
authored
chore: consistent return type among methods (#29)
1 parent 7dc0458 commit c05abde

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/diff_walker.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ impl DiffWalker {
358358

359359
/// Split a schema into multiple schemas, one for each type in the multiple type.
360360
/// Returns the new schema and whether the schema was changed.
361-
fn split_types(schema_object: &mut SchemaObject) -> (&mut SchemaObject, bool) {
361+
fn split_types(schema_object: &mut SchemaObject) -> bool {
362362
let is_split = match schema_object.effective_type() {
363363
InternalJsonSchemaType::Multiple(types)
364364
if schema_object.subschemas().any_of.is_none() =>
@@ -381,7 +381,7 @@ impl DiffWalker {
381381
}
382382
_ => false,
383383
};
384-
(schema_object, is_split)
384+
is_split
385385
}
386386

387387
fn do_diff(
@@ -393,8 +393,8 @@ impl DiffWalker {
393393
rhs: &mut SchemaObject,
394394
) -> Result<(), Error> {
395395
self.resolve_references(lhs, rhs)?;
396-
let (lhs, is_lhs_split) = Self::split_types(lhs);
397-
let (rhs, is_rhs_split) = Self::split_types(rhs);
396+
let is_lhs_split = Self::split_types(lhs);
397+
let is_rhs_split = Self::split_types(rhs);
398398
self.diff_any_of(json_path, is_rhs_split, lhs, rhs)?;
399399
if !comparing_any_of {
400400
self.diff_instance_types(json_path, lhs, rhs);

0 commit comments

Comments
 (0)