mirrors/SFML
1
0
Fork 0
mirror of https://github.com/SFML/SFML.git synced 2024-11-24 20:31:05 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary trailing return types

Browse Source 
The lamdba can deduce the return type correctly having it deducing it
automaticly makes less rooms for mistakes later when we change the impl.
This commit is contained in:
ZXShady 2024-10-08 02:07:04 +01:00 committed by Chris Thrasher
parent e8f59f9d4c
commit d9f3634750
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
include/SFML/Graphics/Color.inl
View File

@ -74,7 +74,7 @@ constexpr bool operator!=(Color left, Color right)
//////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////
constexpr Color operator+(Color left, Color right) constexpr Color operator+(Color left, Color right)
{ {
const auto clampedAdd = [](std::uint8_t lhs, std::uint8_t rhs) -> std::uint8_t const auto clampedAdd = [](std::uint8_t lhs, std::uint8_t rhs)
{ {
const int intResult = int{lhs} + int{rhs}; const int intResult = int{lhs} + int{rhs};
return static_cast<std::uint8_t>(intResult < 255 ? intResult : 255); return static_cast<std::uint8_t>(intResult < 255 ? intResult : 255);
@ -90,7 +90,7 @@ constexpr Color operator+(Color left, Color right)
//////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////
constexpr Color operator-(Color left, Color right) constexpr Color operator-(Color left, Color right)
{ {
const auto clampedSub = [](std::uint8_t lhs, std::uint8_t rhs) -> std::uint8_t const auto clampedSub = [](std::uint8_t lhs, std::uint8_t rhs)
{ {
const int intResult = int{lhs} - int{rhs}; const int intResult = int{lhs} - int{rhs};
return static_cast<std::uint8_t>(intResult > 0 ? intResult : 0); return static_cast<std::uint8_t>(intResult > 0 ? intResult : 0);
@ -106,7 +106,7 @@ constexpr Color operator-(Color left, Color right)
//////////////////////////////////////////////////////////// ////////////////////////////////////////////////////////////
constexpr Color operator*(Color left, Color right) constexpr Color operator*(Color left, Color right)
{ {
const auto scaledMul = [](std::uint8_t lhs, std::uint8_t rhs) -> std::uint8_t const auto scaledMul = [](std::uint8_t lhs, std::uint8_t rhs)
{ {
const auto uint16Result = static_cast<std::uint16_t>(std::uint16_t{lhs} * std::uint16_t{rhs}); const auto uint16Result = static_cast<std::uint16_t>(std::uint16_t{lhs} * std::uint16_t{rhs});
return static_cast<std::uint8_t>(uint16Result / 255u); return static_cast<std::uint8_t>(uint16Result / 255u);