From f902fba76eae90525ca115730c0a46b818aa0a26 Mon Sep 17 00:00:00 2001 From: GreyMerlin Date: Mon, 22 Oct 2018 17:31:14 -0700 Subject: [PATCH] Trivial Bugfix: Debugger generates three warnings **Symptom:** Starting WinMerge with an active Debugger generates three identical warning messages ... `... \dlgdata.cpp(254) : AppMsg - Warning: dialog data checkbox value (205) out of range.` **Solution:** The `m_bReadOnly`array was not being initialized, thus all of the `bool` elements had a value of `0xcd` = 205 when initially given to the three **Read-only** checkboxes on the OpenView dialog. The array is now properly initialized in the `COpenView::` constructor. **Also:** An unrelated trivial comment is repaired. --- Src/CompareEngines/ByteComparator.cpp | 2 +- Src/OpenView.cpp | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Src/CompareEngines/ByteComparator.cpp b/Src/CompareEngines/ByteComparator.cpp index 223178a0f..615c0da0a 100644 --- a/Src/CompareEngines/ByteComparator.cpp +++ b/Src/CompareEngines/ByteComparator.cpp @@ -541,5 +541,5 @@ inline void ByteComparator::HandleSide1Eol(char **ptr, const char *end, bool eof *ptr = pbuf; } -} //namespace namespace CompareEngines +} //namespace CompareEngines diff --git a/Src/OpenView.cpp b/Src/OpenView.cpp index 011e5780f..41c4f5337 100644 --- a/Src/OpenView.cpp +++ b/Src/OpenView.cpp @@ -120,6 +120,7 @@ COpenView::COpenView() , m_pDropHandler(NULL) , m_dwFlags() , m_bAutoCompleteReady() + , m_bReadOnly {false, false, false} { } -- 2.11.0