Moved ownership of hex viewer settings to TargetMemoryInspectionPane and corrected default settings

src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/HexViewerWidget/HexViewerWidget.cpp

@@ -12,6 +12,7 @@ using Bloom::Targets::TargetMemoryDescriptor;
 
 HexViewerWidget::HexViewerWidget(
     const TargetMemoryDescriptor& targetMemoryDescriptor,
+    HexViewerWidgetSettings& settings,
     std::vector<FocusedMemoryRegion>& focusedMemoryRegions,
     std::vector<ExcludedMemoryRegion>& excludedMemoryRegions,
     InsightWorker& insightWorker,
@@ -19,6 +20,7 @@ HexViewerWidget::HexViewerWidget(
 ):
     QWidget(parent),
     targetMemoryDescriptor(targetMemoryDescriptor),
+    settings(settings),
     focusedMemoryRegions(focusedMemoryRegions),
     excludedMemoryRegions(excludedMemoryRegions),
     insightWorker(insightWorker)
@@ -74,9 +76,9 @@ HexViewerWidget::HexViewerWidget(
     );
     this->byteItemGraphicsScene = this->byteItemGraphicsView->getScene();
 
-    this->setStackMemoryHighlightingEnabled(true);
+    this->setStackMemoryHighlightingEnabled(this->settings.highlightStackMemory);
-    this->setHoveredRowAndColumnHighlightingEnabled(true);
+    this->setHoveredRowAndColumnHighlightingEnabled(this->settings.highlightHoveredRowAndCol);
-    this->setFocusedMemoryHighlightingEnabled(true);
+    this->setFocusedMemoryHighlightingEnabled(this->settings.highlightFocusedMemory);
 
     QObject::connect(
         this->highlightStackMemoryButton,

src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/HexViewerWidget/HexViewerWidget.hpp

@@ -31,6 +31,7 @@ namespace Bloom::Widgets
 
         HexViewerWidget(
             const Targets::TargetMemoryDescriptor& targetMemoryDescriptor,
+            HexViewerWidgetSettings& settings,
             std::vector<FocusedMemoryRegion>& focusedMemoryRegions,
             std::vector<ExcludedMemoryRegion>& excludedMemoryRegions,
             InsightWorker& insightWorker,
@@ -52,7 +53,7 @@ namespace Bloom::Widgets
 
         InsightWorker& insightWorker;
 
-        HexViewerWidgetSettings settings = HexViewerWidgetSettings();
+        HexViewerWidgetSettings& settings;
 
         QWidget* container = nullptr;
         QWidget* toolBar = nullptr;

src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/HexViewerWidget/HexViewerWidgetSettings.hpp

@@ -4,9 +4,9 @@ namespace Bloom::Widgets
 {
     struct HexViewerWidgetSettings
     {
-        bool highlightStackMemory = false;
+        bool highlightStackMemory = true;
-        bool highlightFocusedMemory = false;
+        bool highlightFocusedMemory = true;
-        bool highlightHoveredRowAndCol = false;
+        bool highlightHoveredRowAndCol = true;
         bool displayAsciiValues = false;
     };
 }

src/Insight/UserInterfaces/InsightWindow/Widgets/TargetMemoryInspectionPane/TargetMemoryInspectionPane.cpp

@@ -61,6 +61,7 @@ TargetMemoryInspectionPane::TargetMemoryInspectionPane(
     this->manageMemoryRegionsButton = this->container->findChild<SvgToolButton*>("manage-memory-regions-btn");
     this->hexViewerWidget = new HexViewerWidget(
         this->targetMemoryDescriptor,
+        this->settings.hexViewerWidgetSettings,
         this->settings.focusedMemoryRegions,
         this->settings.excludedMemoryRegions,
         this->insightWorker,