From 717ffcd80ce45037cd04ad35565a43f6c3c223bc Mon Sep 17 00:00:00 2001 From: Ilya Ivanov Date: Thu, 27 Apr 2017 15:06:56 +0300 Subject: [PATCH] Fix wrong variable name in assignment in wxRichTextSizePage dtor The "sm_showMinMaxSizeControls" was assigned twice, while "sm_enablePositionAndSizeUnits" was not assigned at all, apparently by mistake. Thanks to PVS-Studio for finding this issue (V519 The 'sm_showMinMaxSizeControls' variable is assigned values twice successively). --- src/richtext/richtextsizepage.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/richtext/richtextsizepage.cpp b/src/richtext/richtextsizepage.cpp index e0a6164b58..0cf3569df1 100644 --- a/src/richtext/richtextsizepage.cpp +++ b/src/richtext/richtextsizepage.cpp @@ -117,7 +117,7 @@ wxRichTextSizePage::~wxRichTextSizePage() sm_showAlignmentControls = true; sm_showFloatingAndAlignmentControls = true; sm_showMinMaxSizeControls = true; - sm_showMinMaxSizeControls = true; + sm_enablePositionAndSizeUnits = true; sm_enablePositionAndSizeCheckboxes = true; ////@begin wxRichTextSizePage destruction