From 0149f7d4e4cdf2dc28ffd5841df5d6aad45d1fad Mon Sep 17 00:00:00 2001 From: Sam Atkins Date: Wed, 11 Dec 2024 15:16:34 +0000 Subject: [PATCH] LibWeb/CSS: Remove unnecessary CalculatedStyleValue const-casts --- Libraries/LibWeb/CSS/CSSStyleValue.cpp | 2 +- Libraries/LibWeb/CSS/StyleProperties.cpp | 12 ++++++------ .../LibWeb/CSS/StyleValues/CalculatedStyleValue.cpp | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/Libraries/LibWeb/CSS/CSSStyleValue.cpp b/Libraries/LibWeb/CSS/CSSStyleValue.cpp index 744137d789..992015e9cf 100644 --- a/Libraries/LibWeb/CSS/CSSStyleValue.cpp +++ b/Libraries/LibWeb/CSS/CSSStyleValue.cpp @@ -407,7 +407,7 @@ int CSSStyleValue::to_font_weight() const return round_to(as_number().number()); } if (is_calculated()) { - auto maybe_weight = const_cast(as_calculated()).resolve_integer(); + auto maybe_weight = as_calculated().resolve_integer(); if (maybe_weight.has_value()) return maybe_weight.value(); } diff --git a/Libraries/LibWeb/CSS/StyleProperties.cpp b/Libraries/LibWeb/CSS/StyleProperties.cpp index 8e1d67ac55..657ef4a8e1 100644 --- a/Libraries/LibWeb/CSS/StyleProperties.cpp +++ b/Libraries/LibWeb/CSS/StyleProperties.cpp @@ -138,7 +138,7 @@ Variant StyleProperties::gap_value(CSS::PropertyID } if (value.is_calculated()) - return LengthPercentage { const_cast(value.as_calculated()) }; + return LengthPercentage { value.as_calculated() }; if (value.is_percentage()) return LengthPercentage { value.as_percentage().percentage() }; @@ -170,7 +170,7 @@ CSS::Size StyleProperties::size_value(CSS::PropertyID id) const } if (value.is_calculated()) - return CSS::Size::make_calculated(const_cast(value.as_calculated())); + return CSS::Size::make_calculated(value.as_calculated()); if (value.is_percentage()) return CSS::Size::make_percentage(value.as_percentage().percentage()); @@ -197,7 +197,7 @@ Optional StyleProperties::length_percentage(CSS::PropertyID id auto const& value = property(id); if (value.is_calculated()) - return LengthPercentage { const_cast(value.as_calculated()) }; + return LengthPercentage { value.as_calculated() }; if (value.is_percentage()) return value.as_percentage().percentage(); @@ -318,7 +318,7 @@ float StyleProperties::resolve_opacity_value(CSSStyleValue const& value) else dbgln("Unable to resolve calc() as opacity (percentage): {}", value.to_string(CSSStyleValue::SerializationMode::Normal)); } else if (calculated.resolves_to_number()) { - auto maybe_number = const_cast(value.as_calculated()).resolve_number(); + auto maybe_number = value.as_calculated().resolve_number(); if (maybe_number.has_value()) unclamped_opacity = maybe_number.value(); else @@ -636,7 +636,7 @@ static Optional length_percentage_for_style_value(CSSStyleValu if (value.is_percentage()) return value.as_percentage().percentage(); if (value.is_calculated()) - return LengthPercentage { const_cast(value.as_calculated()) }; + return LengthPercentage { value.as_calculated() }; return {}; } @@ -1168,7 +1168,7 @@ Variant StyleProperties::vertical_ali return CSS::LengthPercentage(value.as_percentage().percentage()); if (value.is_calculated()) - return LengthPercentage { const_cast(value.as_calculated()) }; + return LengthPercentage { value.as_calculated() }; VERIFY_NOT_REACHED(); } diff --git a/Libraries/LibWeb/CSS/StyleValues/CalculatedStyleValue.cpp b/Libraries/LibWeb/CSS/StyleValues/CalculatedStyleValue.cpp index 0fad75cdcc..d6d0a06063 100644 --- a/Libraries/LibWeb/CSS/StyleValues/CalculatedStyleValue.cpp +++ b/Libraries/LibWeb/CSS/StyleValues/CalculatedStyleValue.cpp @@ -2470,7 +2470,7 @@ bool CalculatedStyleValue::equals(CSSStyleValue const& other) const if (type() != other.type()) return false; - return m_calculation->equals(*static_cast(other).m_calculation); + return m_calculation->equals(*other.as_calculated().m_calculation); } Optional CalculatedStyleValue::resolve_angle() const