Skip to content

Commit 7f2ced5

Browse files
committed
Small optimization refactors.
done as part of CURA-12352
1 parent 0540d55 commit 7f2ced5

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/FffGcodeWriter.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3118,20 +3118,20 @@ bool FffGcodeWriter::processInsets(
31183118
// Create raw speed regions
31193119
const AngleDegrees overhang_step = (90.0 - wall_overhang_angle) / static_cast<double>(overhang_angles_count);
31203120
std::vector<SpeedRegion> speed_regions;
3121-
speed_regions.reserve(overhang_angles_count + 1);
3121+
speed_regions.reserve(overhang_angles_count + 2);
31223122

31233123
constexpr bool dont_chunk_first = false;
3124-
speed_regions.push_back(SpeedRegion{ wall_overhang_angle, 1.0_r, dont_chunk_first }); // Initial internal region, always 100% speed factor
3124+
speed_regions.emplace_back(wall_overhang_angle, 1.0_r, dont_chunk_first); // Initial internal region, always 100% speed factor
31253125

31263126
for (size_t angle_index = 1; angle_index <= overhang_angles_count; ++angle_index)
31273127
{
31283128
const AngleDegrees actual_wall_overhang_angle = wall_overhang_angle + static_cast<double>(angle_index) * overhang_step;
31293129
const Ratio speed_factor = overhang_speed_factors[angle_index - 1];
31303130

3131-
speed_regions.push_back(SpeedRegion{ actual_wall_overhang_angle, speed_factor });
3131+
speed_regions.emplace_back(actual_wall_overhang_angle, speed_factor);
31323132
}
31333133

3134-
speed_regions.push_back(SpeedRegion{ 90.0, overhang_speed_factors.back() }); // Final "everything else" speed region
3134+
speed_regions.emplace_back(90.0, overhang_speed_factors.back()); // Final "everything else" speed region
31353135

31363136
// Now merge regions that have similar speed factors (saves calculations and avoid generating micro-segments)
31373137
auto merged_regions = speed_regions
@@ -3147,7 +3147,7 @@ bool FffGcodeWriter::processInsets(
31473147
for (const auto& regions : merged_regions)
31483148
{
31493149
const SpeedRegion& last_region = *ranges::prev(regions.end());
3150-
overhang_masks.push_back(LayerPlan::OverhangMask{ get_supported_region(last_region.overhang_angle), last_region.speed_factor });
3150+
overhang_masks.emplace_back(get_supported_region(last_region.overhang_angle), last_region.speed_factor);
31513151
}
31523152
}
31533153
}

0 commit comments

Comments
 (0)