From 5fed956d65a92b1e36377b7ab96578f051ca090c Mon Sep 17 00:00:00 2001 From: Simon Sapin Date: Fri, 19 Jun 2020 15:30:23 +0200 Subject: Remove the now single-field `Accumulator` struct --- components/layout_2020/flow/construct.rs | 22 +++++++--------------- 1 file changed, 7 insertions(+), 15 deletions(-) (limited to 'components/layout_2020/flow/construct.rs') diff --git a/components/layout_2020/flow/construct.rs b/components/layout_2020/flow/construct.rs index 0f49fb85422..d4151e13c51 100644 --- a/components/layout_2020/flow/construct.rs +++ b/components/layout_2020/flow/construct.rs @@ -198,15 +198,10 @@ impl BlockContainer { builder.end_ongoing_inline_formatting_context(); } - struct Accumulator { - contains_floats: ContainsFloats, - } - let mut acc = Accumulator { - contains_floats: builder.contains_floats, - }; - let mapfold = |acc: &mut Accumulator, creator: BlockLevelJob<'dom, _>| { + let mut contains_floats = builder.contains_floats; + let mapfold = |contains_floats: &mut ContainsFloats, creator: BlockLevelJob<'dom, _>| { let (block_level_box, box_contains_floats) = creator.finish(context); - acc.contains_floats |= box_contains_floats; + *contains_floats |= box_contains_floats; block_level_box }; let block_level_boxes = if context.use_rayon { @@ -214,13 +209,11 @@ impl BlockContainer { .block_level_boxes .into_par_iter() .mapfold_reduce_into( - &mut acc, + &mut contains_floats, mapfold, - || Accumulator { - contains_floats: ContainsFloats::No, - }, + || ContainsFloats::No, |left, right| { - left.contains_floats |= right.contains_floats; + *left |= right; }, ) .collect() @@ -228,12 +221,11 @@ impl BlockContainer { builder .block_level_boxes .into_iter() - .map(|x| mapfold(&mut acc, x)) + .map(|x| mapfold(&mut contains_floats, x)) .collect() }; let container = BlockContainer::BlockLevelBoxes(block_level_boxes); - let Accumulator { contains_floats } = acc; (container, contains_floats) } } -- cgit v1.2.3