From 1057d5f23a435c8903763b79db383cf6803e8a98 Mon Sep 17 00:00:00 2001
From: RKrom <robin@getgreenshot.org>
Date: Wed, 6 Mar 2013 19:31:07 +0000
Subject: [PATCH] Fixed [#1469] by checking if there are grippers.

git-svn-id: http://svn.code.sf.net/p/greenshot/code/trunk@2516 7dccd23d-a4a3-4e1f-8c07-b4c1b4018ab4
---
 Greenshot/Drawing/DrawableContainer.cs | 24 +++++++++++++++++-------
 Greenshot/Drawing/FreehandContainer.cs |  8 +++++---
 Greenshot/Drawing/LineContainer.cs     |  6 ++++--
 3 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/Greenshot/Drawing/DrawableContainer.cs b/Greenshot/Drawing/DrawableContainer.cs
index 837d9412c..3f2b7d144 100644
--- a/Greenshot/Drawing/DrawableContainer.cs
+++ b/Greenshot/Drawing/DrawableContainer.cs
@@ -323,6 +323,9 @@ namespace Greenshot.Drawing {
 		}
 		
 		protected virtual void DoLayout() {
+			if (grippers == null) {
+				return;
+			}
 			if (!layoutSuspended) {
 				int[] xChoords = new int[]{this.Left-2,this.Left+this.Width/2-2,this.Left+this.Width-2};
 				int[] yChoords = new int[]{this.Top-2,this.Top+this.Height/2-2,this.Top+this.Height-2};
@@ -471,17 +474,24 @@ namespace Greenshot.Drawing {
 		}
 		
 		public virtual void ShowGrippers() {
-			for (int i=0; i<grippers.Length; i++) {
-				if(grippers[i].Enabled) grippers[i].Show();
-				else grippers[i].Hide();
+			if (grippers != null) {
+				for (int i = 0; i < grippers.Length; i++) {
+					if (grippers[i].Enabled) {
+						grippers[i].Show();
+					} else {
+						grippers[i].Hide();
+					}
+				}
 			}
 			this.ResumeLayout();
 		}
 		
 		public void HideGrippers() {
 			this.SuspendLayout();
-			for (int i=0; i<grippers.Length; i++) {
-				grippers[i].Hide();
+			if (grippers != null) {
+				for (int i = 0; i < grippers.Length; i++) {
+					grippers[i].Hide();
+				}
 			}
 		}
 		
@@ -554,11 +564,11 @@ namespace Greenshot.Drawing {
 		}
 		
 		private void SwitchParent(Surface newParent) {
-			if (parent != null) {
+			if (parent != null && grippers != null) {
 				for (int i=0; i<grippers.Length; i++) {
 					parent.Controls.Remove(grippers[i]);
 				}
-			} else if(grippers == null) {
+			} else if (grippers == null) {
 				InitControls();
 			}
 			parent = newParent;
diff --git a/Greenshot/Drawing/FreehandContainer.cs b/Greenshot/Drawing/FreehandContainer.cs
index 12e59bfc4..83c92663c 100644
--- a/Greenshot/Drawing/FreehandContainer.cs
+++ b/Greenshot/Drawing/FreehandContainer.cs
@@ -58,9 +58,11 @@ namespace Greenshot.Drawing {
 		}
 		
 		protected void Init() {
-			for(int i=0; i<grippers.Length; i++) {
-				grippers[i].Enabled = false;
-				grippers[i].Visible = false;
+			if (grippers != null) {
+				for (int i = 0; i < grippers.Length; i++) {
+					grippers[i].Enabled = false;
+					grippers[i].Visible = false;
+				}
 			}
 		}
 		
diff --git a/Greenshot/Drawing/LineContainer.cs b/Greenshot/Drawing/LineContainer.cs
index cfc5299f8..13f6359ac 100644
--- a/Greenshot/Drawing/LineContainer.cs
+++ b/Greenshot/Drawing/LineContainer.cs
@@ -50,8 +50,10 @@ namespace Greenshot.Drawing {
 		}
 
 		protected void Init() {
-			foreach(int index in new int[]{1,2,3,5,6,7}) {
-				grippers[index].Enabled = false;
+			if (grippers != null) {
+				foreach (int index in new int[] { 1, 2, 3, 5, 6, 7 }) {
+					grippers[index].Enabled = false;
+				}
 			}
 		}