Prefer one-argument constructor

rism-digital · Jan 25, 2025 · 1dda133 · 1dda133
1 parent a6ac88a
commit 1dda133
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 11 deletions.
diff --git a/src/convertfunctor.cpp b/src/convertfunctor.cpp
@@ -965,8 +965,8 @@ void ConvertToCmnFunctor::SplitDurationIntoCmn(
     const int semiBrevisDots = (prolatioMajor) ? 1 : 0;
     const int brevisDots = (tempusPerfectum) ? 1 : 0;
 
-    const Fraction semiBrevis = Fraction(1, 1) * abs(mensur->GetProlatio()) / 2;
-    const Fraction brevis = Fraction(1, 1) * abs(mensur->GetTempus());
+    const Fraction semiBrevis = Fraction(1) * abs(mensur->GetProlatio()) / 2;
+    const Fraction brevis = Fraction(1) * abs(mensur->GetTempus());
 
     // First see if we are expecting a breve and if the duration is long enough
     if (elementDur == DURATION_breve) {

diff --git a/src/durationinterface.cpp b/src/durationinterface.cpp
@@ -100,7 +100,7 @@ Fraction DurationInterface::GetInterfaceAlignmentMensuralDuration(
 
     if (!currentMensur) {
         LogWarning("No current mensur for calculating duration");
-        return Fraction(1, 1);
+        return Fraction(1);
     }
 
     if (this->HasNum() || this->HasNumbase()) {

diff --git a/src/horizontalaligner.cpp b/src/horizontalaligner.cpp
@@ -516,15 +516,15 @@ void Alignment::Reset()
     Object::Reset();
 
     m_xRel = 0;
-    m_time = Fraction(0, 1);
+    m_time = Fraction(0);
     m_type = ALIGNMENT_DEFAULT;
 
-    ClearGraceAligners();
+    this->ClearGraceAligners();
 }
 
 Alignment::~Alignment()
 {
-    ClearGraceAligners();
+    this->ClearGraceAligners();
 }
 
 void Alignment::ClearGraceAligners()

diff --git a/src/layerelement.cpp b/src/layerelement.cpp
@@ -675,7 +675,7 @@ Fraction LayerElement::GetAlignmentDuration(
     const AlignMeterParams &params, bool notGraceOnly, data_NOTATIONTYPE notationType) const
 {
     if (this->IsGraceNote() && notGraceOnly) {
-        return Fraction(0, 1);
+        return Fraction(0);
     }
 
     // Mensural chords are aligned looking at the duration of the notes
@@ -777,7 +777,7 @@ Fraction LayerElement::GetAlignmentDuration(
         return (syllable->GetLast() == this) ? NEUME_MEDIUM_SPACE : NEUME_SMALL_SPACE;
     }
     else {
-        return Fraction(0, 1);
+        return Fraction(0);
     }
 }
 
@@ -791,7 +791,7 @@ Fraction LayerElement::GetSameAsContentAlignmentDuration(
     const AlignMeterParams &params, bool notGraceOnly, data_NOTATIONTYPE notationType) const
 {
     if (!this->HasSameasLink() || !this->GetSameasLink()->Is({ BEAM, FTREM, TUPLET })) {
-        return Fraction(0, 1);
+        return Fraction(0);
     }
 
     const LayerElement *sameas = vrv_cast<const LayerElement *>(this->GetSameasLink());
@@ -804,7 +804,7 @@ Fraction LayerElement::GetContentAlignmentDuration(
     const AlignMeterParams &params, bool notGraceOnly, data_NOTATIONTYPE notationType) const
 {
     if (!this->Is({ BEAM, FTREM, TUPLET })) {
-        return Fraction(0, 1);
+        return Fraction(0);
     }
 
     Fraction duration;

diff --git a/src/view_element.cpp b/src/view_element.cpp
@@ -1166,7 +1166,7 @@ void View::DrawMRest(DeviceContext *dc, LayerElement *element, Layer *layer, Sta
 
     const bool drawingCueSize = mRest->GetDrawingCueSize();
     int x = mRest->GetDrawingX();
-    const bool isDouble = (measure->m_measureAligner.GetMaxTime() >= Fraction(2, 1));
+    const bool isDouble = (measure->m_measureAligner.GetMaxTime() >= Fraction(2));
     int y = isDouble ? element->GetDrawingY() - m_doc->GetDrawingDoubleUnit(staffSize) : element->GetDrawingY();
     char32_t rest = isDouble ? SMUFL_E4E2_restDoubleWhole : SMUFL_E4E3_restWhole;