Performance improvement

Author: mike-spa

src/engraving/libmscore/beam.cpp

@@ -2513,9 +2513,6 @@ Shape Beam::shape() const
 // BEAM SEGMENT CLASS
 //-------------------------------------------------------
 
-BeamSegment::BeamSegment(Beam* b)
-    : EngravingItem(ElementType::BEAM_SEGMENT, b), beam(b) {}
-
 Shape BeamSegment::shape() const
 {
     Shape shape;
@@ -2531,7 +2528,7 @@ Shape BeamSegment::shape() const
     if (RealIsEqual(startPoint.y(), endPoint.y())) {
         RectF rect(startPoint.x(), startPoint.y(), beamHorizontalLength, _beamWidth / 2);
         rect.adjust(0.0, -_beamWidth / 2, 0.0, 0.0);
-        shape.add(rect, this);
+        shape.add(rect, beam);
         return shape;
     }
     // If not, break the beam shape into multiple rectangles
@@ -2548,7 +2545,7 @@ Shape BeamSegment::shape() const
     for (PointF point : pointsOnBeamLine) {
         RectF rect(point.x(), point.y(), horizontalStep, _beamWidth / 2);
         rect.adjust(0.0, -_beamWidth / 2, 0.0, 0.0);
-        shape.add(rect, this);
+        shape.add(rect, beam);
     }
     return shape;
 }

src/engraving/libmscore/beam.h

@@ -39,8 +39,9 @@ enum class SpannerSegmentType;
 
 struct BeamFragment;
 
-class BeamSegment : public EngravingItem
+class BeamSegment
 {
+    OBJECT_ALLOCATOR(engraving, BeamSegment)
 public:
     mu::LineF line;
     int level;
@@ -50,11 +51,11 @@ class BeamSegment : public EngravingItem
     bool isBeamlet = false;
     bool isBefore = false;
 
-    Shape shape() const override;
+    Shape shape() const;
     Beam* beam;
 
-    BeamSegment(Beam* b);
-    BeamSegment* clone() const override { return new BeamSegment(*this); }
+    BeamSegment(Beam* b)
+        : beam(b) {}
 };
 
 //---------------------------------------------------------

src/engraving/libmscore/score.cpp

@@ -5919,7 +5919,7 @@ void Score::createPaddingTable()
     }
 
     // This is needed for beamlets, not beams themselves
-    _paddingTable[ElementType::BEAM_SEGMENT][ElementType::BEAM_SEGMENT] = 0.4 * spatium();
+    _paddingTable[ElementType::BEAM][ElementType::BEAM] = 0.4 * spatium();
 }
 
 //--------------------------------------------------------