diff --git a/src/CalcViewModel/Common/CopyPasteManager.cpp b/src/CalcViewModel/Common/CopyPasteManager.cpp index 36658cdc..a03350e8 100644 --- a/src/CalcViewModel/Common/CopyPasteManager.cpp +++ b/src/CalcViewModel/Common/CopyPasteManager.cpp @@ -418,8 +418,7 @@ bool CopyPasteManager::TryOperandToULL(const wstring& operand, int numberBase, u return false; } - // Default to base10 - int intBase = 10; + int intBase; switch (numberBase) { case HexBase: @@ -431,7 +430,8 @@ bool CopyPasteManager::TryOperandToULL(const wstring& operand, int numberBase, u case BinBase: intBase = 2; break; - case DecBase: // Unneeded as the default is already 10? + default: + case DecBase: intBase = 10; break; } diff --git a/src/CalcViewModel/Common/CopyPasteManager.h b/src/CalcViewModel/Common/CopyPasteManager.h index 79dd73d9..3ea93e80 100644 --- a/src/CalcViewModel/Common/CopyPasteManager.h +++ b/src/CalcViewModel/Common/CopyPasteManager.h @@ -13,7 +13,6 @@ namespace CalculatorUnitTests namespace CalculatorApp { - // TODO: Could be made into an enum : unsigned ? constexpr auto QwordType = 1; constexpr auto DwordType = 2; constexpr auto WordType = 3;