From b8e5e22d113abc69393fb84858131d26f8735b07 Mon Sep 17 00:00:00 2001 From: Nicholas Baron Date: Tue, 16 Apr 2019 23:26:13 -0700 Subject: [PATCH] Removed unneeded comments --- src/CalcViewModel/Common/CopyPasteManager.cpp | 6 +++--- src/CalcViewModel/Common/CopyPasteManager.h | 1 - 2 files changed, 3 insertions(+), 4 deletions(-) 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;