From 4658ee0f18469fa471ade6bdb5cdc0681b3d0b83 Mon Sep 17 00:00:00 2001
From: "andres.delikat" <andres.delikat@4bbf4f03-6f44-549e-7e59-35d4b0962838>
Date: Mon, 7 Mar 2011 03:23:05 +0000
Subject: [PATCH] Hex editor/Memory Viewer - More code refactoring

---
 BizHawk.MultiClient/tools/HexEditor.cs    | 4 ----
 BizHawk.MultiClient/tools/MemoryViewer.cs | 3 +--
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/BizHawk.MultiClient/tools/HexEditor.cs b/BizHawk.MultiClient/tools/HexEditor.cs
index 76f2f7b058..76750c0259 100644
--- a/BizHawk.MultiClient/tools/HexEditor.cs
+++ b/BizHawk.MultiClient/tools/HexEditor.cs
@@ -72,8 +72,6 @@ namespace BizHawk.MultiClient
         {
             SetMemoryDomainMenu(); //Calls update routines
             MemoryViewer.ResetScrollBar();
-            MemoryViewer.SetUpScrollBar();
-            MemoryViewer.Refresh();
         }
 
         private void restoreWindowSizeToolStripMenuItem_Click(object sender, EventArgs e)
@@ -99,8 +97,6 @@ namespace BizHawk.MultiClient
             }
             UpdateDomainString();
             MemoryViewer.ResetScrollBar();
-            MemoryViewer.SetUpScrollBar();
-            MemoryViewer.Refresh();
         }
 
         private void UpdateDomainString()
diff --git a/BizHawk.MultiClient/tools/MemoryViewer.cs b/BizHawk.MultiClient/tools/MemoryViewer.cs
index d83e2f23a3..213432a5b8 100644
--- a/BizHawk.MultiClient/tools/MemoryViewer.cs
+++ b/BizHawk.MultiClient/tools/MemoryViewer.cs
@@ -9,8 +9,6 @@ namespace BizHawk.MultiClient
 {
     public class MemoryViewer : GroupBox
     {
-        //TODO: refactor so parent never needs to call setscrollbars (put in reset method?)
-        //think about scrollbar = 0, there needs to be a way for the parent to set that separately from scrollbars
         //data size variable, and adjust drawing based on it, and a method for parent to set it
 
         public VScrollBar vScrollBar1;
@@ -80,6 +78,7 @@ namespace BizHawk.MultiClient
         public void ResetScrollBar()
         {
             vScrollBar1.Value = 0;
+            SetUpScrollBar();
             Refresh();
         }