VOID_Core: Added a module.LoadConfig call to LoadModule.
VOID_Core: Added a module.LoadConfig call to LoadModule.

--- /dev/null
+++ b/AVOID_Configurable.cs
@@ -1,1 +1,29 @@
+//
+//  AVOID_Configurable.cs
+//
+//  Author:
+//       toadicus <>
+//
+//  Copyright (c) 2013 toadicus
+//
+//  This program is free software: you can redistribute it and/or modify
+//  it under the terms of the GNU General Public License as published by
+//  the Free Software Foundation, either version 3 of the License, or
+//  (at your option) any later version.
+//
+//  This program is distributed in the hope that it will be useful,
+//  but WITHOUT ANY WARRANTY; without even the implied warranty of
+//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+//  GNU General Public License for more details.
+//
+//  You should have received a copy of the GNU General Public License
+//  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+using System;
 
+namespace VOID
+{
+	[AttributeUsage(AttributeTargets.Method)]
+	public class AVOID_Configurable : Attribute { }
+}
+
+

--- a/IVOID_Module.cs
+++ b/IVOID_Module.cs
@@ -24,13 +24,16 @@
 {
 	public interface IVOID_Module
 	{
-		bool hasGUIConfig { get; }
-		bool toggleActive { get; }
+		string Name { get; }
+		bool hasConfigurables { get; }
+		bool toggleActive { get; set; }
 		bool guiRunning { get; }
 
 		void DrawGUI();
 		void StartGUI();
 		void StopGUI();
+
+		void DrawConfigurables();
 
 		void LoadConfig();
 

file:a/VOID_ConfigValue.cs (deleted)
--- a/VOID_ConfigValue.cs
+++ /dev/null
@@ -1,115 +1,1 @@
-//
-//  VOID_Config.cs
-//
-//  Author:
-//       toadicus <>
-//
-//  Copyright (c) 2013 toadicus
-//
-//  This program is free software: you can redistribute it and/or modify
-//  it under the terms of the GNU General Public License as published by
-//  the Free Software Foundation, either version 3 of the License, or
-//  (at your option) any later version.
-//
-//  This program is distributed in the hope that it will be useful,
-//  but WITHOUT ANY WARRANTY; without even the implied warranty of
-//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-//  GNU General Public License for more details.
-//
-//  You should have received a copy of the GNU General Public License
-//  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-using System;
-using System.Collections.Generic;
-using KSP;
-using UnityEngine;
 
-namespace VOID
-{
-	public struct VOID_ConfigValue<T> : IVOID_ConfigValue
-	{
-		private T _value;
-		private Type _type;
-
-		public T value
-		{
-			get
-			{
-				return this._value;
-			}
-			set
-			{
-				this._value = value;
-			}
-		}
-
-		public Type type
-		{
-			get
-			{
-				return this._type;
-			}
-			set
-			{
-				this._type = value;
-			}
-		}
-
-		public object AsType
-		{
-			get
-			{
-				return (T)this._value;
-			}
-		}
-
-		public void SetValue(object v)
-		{
-			this._value = (T)v;
-		}
-
-		public static implicit operator T(VOID_ConfigValue<T> v)
-		{
-			return v.value;
-		}
-
-		public static implicit operator VOID_ConfigValue<T>(T v)
-		{
-			VOID_ConfigValue<T> r = new VOID_ConfigValue<T>();
-			r.value = v;
-			r.type = v.GetType();
-			if (VOID_Core.Initialized)
-			{
-				VOID_Core.Instance.configDirty = true;
-			}
-			return r;
-		}
-	}
-
-	public interface IVOID_ConfigValue
-	{
-		Type type { get; }
-		object AsType { get; }
-		void SetValue(object v);
-	}
-
-	[AttributeUsage(AttributeTargets.Field)]
-	public class AVOID_ConfigValue : Attribute
-	{
-		protected string _name;
-
-		public string Name
-		{
-			get
-			{
-				return this._name;
-			}
-		}
-
-		public AVOID_ConfigValue(string fieldName)
-		{
-			this._name = fieldName;
-		}
-	}
-}
-
-

--- a/VOID_Core.cs
+++ b/VOID_Core.cs
@@ -21,6 +21,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Reflection;
 using KSP;
 using UnityEngine;
 
@@ -60,25 +61,28 @@
 		protected string VoidName = "VOID";
 		protected string VoidVersion = "0.9.9";
 
-		[AVOID_ConfigValue("configValue")]
-		protected VOID_ConfigValue<int> configVersion = 1;
-
-		protected List<VOID_Module> _modules = new List<VOID_Module>();
-
-		[AVOID_ConfigValue("mainWindowPos")]
-		protected VOID_ConfigValue<Rect> mainWindowPos = new Rect(Screen.width / 2, Screen.height / 2, 10f, 10f);
-
-		[AVOID_ConfigValue("mainGuiMinimized")]
-		protected VOID_ConfigValue<bool> mainGuiMinimized = false;
-
-		[AVOID_ConfigValue("configWindowPos")]
-		protected VOID_ConfigValue<Rect> configWindowPos = new Rect(Screen.width / 2, Screen.height  /2, 10f, 10f);
-
-		[AVOID_ConfigValue("configWindowMinimized")]
-		protected VOID_ConfigValue<bool> configWindowMinimized = true;
-
-		[AVOID_ConfigValue("VOIDIconPos")]
-		protected VOID_ConfigValue<Rect> VOIDIconPos = new Rect(Screen.width / 2 - 200, Screen.height - 30, 30f, 30f);
+		[AVOID_SaveValue("configValue")]
+		protected VOID_SaveValue<int> configVersion = 1;
+
+		protected List<IVOID_Module> _modules = new List<IVOID_Module>();
+		protected bool _modulesLoaded = false;
+
+		protected List<Callback> _configurableCallbacks = new List<Callback>();
+
+		[AVOID_SaveValue("mainWindowPos")]
+		protected VOID_SaveValue<Rect> mainWindowPos = new Rect(Screen.width / 2, Screen.height / 2, 10f, 10f);
+
+		[AVOID_SaveValue("mainGuiMinimized")]
+		protected VOID_SaveValue<bool> mainGuiMinimized = false;
+
+		[AVOID_SaveValue("configWindowPos")]
+		protected VOID_SaveValue<Rect> configWindowPos = new Rect(Screen.width / 2, Screen.height  /2, 10f, 10f);
+
+		[AVOID_SaveValue("configWindowMinimized")]
+		protected VOID_SaveValue<bool> configWindowMinimized = true;
+
+		[AVOID_SaveValue("VOIDIconPos")]
+		protected VOID_SaveValue<Rect> VOIDIconPos = new Rect(Screen.width / 2 - 200, Screen.height - 30, 30f, 30f);
 		protected Texture2D VOIDIconOff = new Texture2D(30, 30, TextureFormat.ARGB32, false);
 		protected Texture2D VOIDIconOn = new Texture2D(30, 30, TextureFormat.ARGB32, false);
 		protected Texture2D VOIDIconTexture;
@@ -87,31 +91,31 @@
 
 		protected int windowBaseID = -96518722;
 
-		[AVOID_ConfigValue("togglePower")]
-		public VOID_ConfigValue<bool> togglePower = true;
+		[AVOID_SaveValue("togglePower")]
+		public VOID_SaveValue<bool> togglePower = true;
 
 		public bool powerAvailable = true;
 
-		[AVOID_ConfigValue("consumeResource")]
-		protected VOID_ConfigValue<bool> consumeResource = false;
-
-		[AVOID_ConfigValue("resourceName")]
-		protected VOID_ConfigValue<string> resourceName = "ElectricCharge";
-
-		[AVOID_ConfigValue("resourceRate")]
-		protected VOID_ConfigValue<float> resourceRate = 0.2f;
+		[AVOID_SaveValue("consumeResource")]
+		protected VOID_SaveValue<bool> consumeResource = false;
+
+		[AVOID_SaveValue("resourceName")]
+		protected VOID_SaveValue<string> resourceName = "ElectricCharge";
+
+		[AVOID_SaveValue("resourceRate")]
+		protected VOID_SaveValue<float> resourceRate = 0.2f;
 
 		public float saveTimer = 0;
 
 		protected string defaultSkin = "KSP window 2";
-		protected VOID_ConfigValue<string> _skin;
+		protected VOID_SaveValue<string> _skin;
 
 		public bool configDirty;
 
 		/*
 		 * Properties
 		 * */
-		public List<VOID_Module> Modules
+		public List<IVOID_Module> Modules
 		{
 			get
 			{
@@ -149,27 +153,61 @@
 			this.VOIDIconOn = GameDatabase.Instance.GetTexture (this.VOIDIconOnPath, false);
 			this.VOIDIconOff = GameDatabase.Instance.GetTexture (this.VOIDIconOffPath, false);
 
-			// HACK: This is modular but not extensible.  We need to look outside our assembly or move this to modules.
-			foreach (Type T in System.Reflection.Assembly.GetExecutingAssembly().GetTypes())
-			{
-				Tools.PostDebugMessage (string.Format ("VOID_Core: Testing type {0}", T.Name));
-				if (typeof(IVOID_Module).IsAssignableFrom(T) &&
-				    !T.IsAbstract  &&
-				    !typeof(VOID_Core).IsAssignableFrom(T))
-				{
-					this.LoadModule (T);
-					Tools.PostDebugMessage(string.Format("VOID_Core: Found module {0}.", T.Name));
-				}
-			}
+			this.LoadConfig ();
+		}
+
+		protected void LoadModules()
+		{
+			var types = AssemblyLoader.loadedAssemblies
+				.Select (a => a.assembly.GetExportedTypes ())
+					.SelectMany (t => t)
+					.Where (v => typeof(IVOID_Module).IsAssignableFrom (v)
+					        && !(v.IsInterface || v.IsAbstract) &&
+					        !typeof(VOID_Core).IsAssignableFrom (v)
+					        );
+
+			Tools.PostDebugMessage (string.Format (
+				"{0}: Found {1} modules to check.",
+				this.GetType ().Name,
+				types.Count ()
+				));
+			foreach (var voidType in types)
+			{
+				Tools.PostDebugMessage (string.Format (
+					"{0}: found Type {1}",
+					this.GetType ().Name,
+					voidType.Name
+					));
+
+				this.LoadModule(voidType);
+			}
+
+			this._modulesLoaded = true;
 
 			Tools.PostDebugMessage (string.Format ("VOID_Core: Loaded {0} modules.", this.Modules.Count));
-
-			this.LoadConfig ();
-		}
-
-		public void LoadModule(Type T)
-		{
-			this._modules.Add (Activator.CreateInstance (T) as VOID_Module);
+		}
+
+		protected void LoadModule(Type T)
+		{
+			var existingModules = this._modules.Where (mod => mod.GetType ().Name == T.Name);
+			if (existingModules.Any())
+			{
+				Tools.PostDebugMessage(string.Format(
+					"{0}: refusing to load {1}: already loaded",
+					this.GetType().Name,
+					T.Name
+				));
+				return;
+			}
+			IVOID_Module module = Activator.CreateInstance (T) as IVOID_Module;
+			module.LoadConfig();
+			this._modules.Add (module);
+
+			Tools.PostDebugMessage(string.Format(
+				"{0}: loaded module {1}.",
+				this.GetType().Name,
+				T.Name
+			));
 		}
 
 		public void Update()
@@ -186,7 +224,7 @@
 				this.StopGUI ();
 			}
 
-			foreach (VOID_Module module in this.Modules)
+			foreach (IVOID_Module module in this.Modules)
 			{
 				if (!module.guiRunning && module.toggleActive)
 				{
@@ -236,7 +274,7 @@
 				if (GUILayout.Button("Power " + str)) togglePower = !togglePower;
 			    if (togglePower)
 			    {
-					foreach (VOID_Module module in this.Modules)
+					foreach (IVOID_Module module in this.Modules)
 					{
 						module.toggleActive = GUILayout.Toggle (module.toggleActive, module.Name);
 					}
@@ -260,19 +298,34 @@
 		{
 			GUILayout.BeginVertical ();
 
-			this.consumeResource = GUILayout.Toggle (this.consumeResource, "Consume Resources");
+			this.DrawConfigurables ();
 
 			GUILayout.EndVertical ();
 			GUI.DragWindow ();
 		}
 
+		public override void DrawConfigurables()
+		{
+			this.consumeResource = GUILayout.Toggle (this.consumeResource, "Consume Resources");
+
+			foreach (IVOID_Module mod in this.Modules)
+			{
+				mod.DrawConfigurables ();
+			}
+		}
+
 		public override void DrawGUI()
 		{
+			if (!this._modulesLoaded)
+			{
+				this.LoadModules ();
+			}
+
 			GUI.skin = this.Skin;
 
 			int windowID = this.windowBaseID;
 
-            this.VOIDIconTexture = this.VOIDIconOff;  //icon off default
+			this.VOIDIconTexture = this.VOIDIconOff;  //icon off default
 			if (this.togglePower) this.VOIDIconTexture = this.VOIDIconOn;     //or on if power_toggle==true
 			if (GUI.Button(new Rect(VOIDIconPos), VOIDIconTexture, new GUIStyle()))
 			{
@@ -287,12 +340,12 @@
 					++windowID,
 					_mainWindowPos,
 					this.VOIDMainWindow,
-					string.Join (" ", this.VoidName, this.VoidVersion),
+					string.Join (" ", new string[] {this.VoidName, this.VoidVersion}),
 					GUILayout.Width (250),
 					GUILayout.Height (50)
 				);
 
-				if (_mainWindowPos != this.mainWindowPos.value)
+				if (_mainWindowPos != this.mainWindowPos)
 				{
 					this.mainWindowPos = _mainWindowPos;
 				}
@@ -302,18 +355,18 @@
 			{
 				Rect _configWindowPos = this.configWindowPos;
 
-				this.configWindowPos = GUILayout.Window (
+				_configWindowPos = GUILayout.Window (
 					++windowID,
 					_configWindowPos,
 					this.VOIDConfigWindow,
-					string.Join (" ", this.VoidName, "Configuration"),
+					string.Join (" ", new string[] {this.VoidName, "Configuration"}),
 					GUILayout.Width (250),
 					GUILayout.Height (50)
 				);
 
-				if (_configWindowPos != this.configWindowPos.value)
-				{
-					this.mainWindowPos = _configWindowPos;
+				if (_configWindowPos != this.configWindowPos)
+				{
+					this.configWindowPos = _configWindowPos;
 				}
 			}
 		}
@@ -322,7 +375,7 @@
 		{
 			base.LoadConfig ();
 
-			foreach (VOID_Module module in this.Modules)
+			foreach (IVOID_Module module in this.Modules)
 			{
 				module.LoadConfig ();
 			}
@@ -337,7 +390,7 @@
 
 			base.SaveConfig ();
 
-			foreach (VOID_Module module in this.Modules)
+			foreach (IVOID_Module module in this.Modules)
 			{
 				module.SaveConfig ();
 			}

--- a/VOID_HUD.cs
+++ b/VOID_HUD.cs
@@ -32,12 +32,14 @@
 		/*
 		 * Fields
 		 * */
+		protected new bool _hasConfigurables = true;
+
+		[AVOID_SaveValue("colorIndex")]
+		protected VOID_SaveValue<int> _colorIndex = 0;
+
 		protected List<Color> textColors = new List<Color>();
 
 		protected GUIStyle labelStyle;
-
-		[AVOID_ConfigValue("colorIndex")]
-		protected VOID_ConfigValue<int> _colorIndex = 0;
 
 		protected Vessel vessel = null;
 
@@ -135,6 +137,14 @@
 				GUI.Label (new Rect ((Screen.width * .625f), 0, 300f, 70f), "-- POWER LOST --", labelStyle);
 			}
 		}
+
+		public override void DrawConfigurables()
+		{
+			if (GUILayout.Button ("Change HUD color", GUILayout.ExpandWidth (false)))
+			{
+				++this.ColorIndex;
+			}
+		}
 	}
 }
 

--- a/VOID_Module.cs
+++ b/VOID_Module.cs
@@ -25,26 +25,26 @@
 
 namespace VOID
 {
-	public abstract class VOID_Module
+	public abstract class VOID_Module : IVOID_Module
 	{
 		/*
 		 * Fields
 		 * */
-		[AVOID_ConfigValue("Active")]
-		protected VOID_ConfigValue<bool> _Active = true;
+		[AVOID_SaveValue("Active")]
+		protected VOID_SaveValue<bool> _Active = true;
 		protected bool _Running = false;
-		protected bool _hasGUICfg = false;
+		protected bool _hasConfigurables = false;
 
 		protected string _Name;
 
 		/*
 		 * Properties
 		 * */
-		public virtual bool hasGUIConfig
+		public virtual bool hasConfigurables
 		{
 			get
 			{
-				return this._hasGUICfg;
+				return this._hasConfigurables;
 			}
 		}
 
@@ -100,6 +100,8 @@
 
 		public abstract void DrawGUI();
 
+		public virtual void DrawConfigurables() {}
+
 		public virtual void LoadConfig()
 		{
 			var config = KSP.IO.PluginConfiguration.CreateForType<VOID_Core> ();
@@ -111,22 +113,22 @@
 				BindingFlags.Instance
 				))
 			{
-				object[] attrs = field.GetCustomAttributes(typeof(AVOID_ConfigValue), false);
+				object[] attrs = field.GetCustomAttributes(typeof(AVOID_SaveValue), false);
 
 				if (attrs.Length == 0) {
 					return;
 				}
 
-				AVOID_ConfigValue attr = attrs.FirstOrDefault () as AVOID_ConfigValue;
+				AVOID_SaveValue attr = attrs.FirstOrDefault () as AVOID_SaveValue;
 
 				string fieldName = string.Format("{0}_{1}", this.GetType().Name, attr.Name);
 
 				object fieldValue = field.GetValue(this);
 
 				bool convertBack = false;
-				if (fieldValue is IVOID_ConfigValue)
+				if (fieldValue is IVOID_SaveValue)
 				{
-					fieldValue = (fieldValue as IVOID_ConfigValue).AsType;
+					fieldValue = (fieldValue as IVOID_SaveValue).AsType;
 					convertBack = true;
 				}
 
@@ -134,8 +136,8 @@
 
 				if (convertBack)
 				{
-					Type type = typeof(VOID_ConfigValue<>).MakeGenericType (fieldValue.GetType ());
-					IVOID_ConfigValue convertValue = Activator.CreateInstance (type) as IVOID_ConfigValue;
+					Type type = typeof(VOID_SaveValue<>).MakeGenericType (fieldValue.GetType ());
+					IVOID_SaveValue convertValue = Activator.CreateInstance (type) as IVOID_SaveValue;
 					convertValue.SetValue (fieldValue);
 					fieldValue = convertValue;
 				}
@@ -162,21 +164,21 @@
 				BindingFlags.Instance
 				))
 			{
-				object[] attrs = field.GetCustomAttributes(typeof(AVOID_ConfigValue), false);
+				object[] attrs = field.GetCustomAttributes(typeof(AVOID_SaveValue), false);
 
 				if (attrs.Length == 0) {
 					return;
 				}
 
-				AVOID_ConfigValue attr = attrs.FirstOrDefault () as AVOID_ConfigValue;
+				AVOID_SaveValue attr = attrs.FirstOrDefault () as AVOID_SaveValue;
 
 				string fieldName = string.Format("{0}_{1}", this.GetType().Name, attr.Name);
 
 				object fieldValue = field.GetValue(this);
 
-				if (fieldValue is IVOID_ConfigValue)
+				if (fieldValue is IVOID_SaveValue)
 				{
-					fieldValue = (fieldValue as IVOID_ConfigValue).AsType;
+					fieldValue = (fieldValue as IVOID_SaveValue).AsType;
 				}
 
 				config.SetValue(fieldName, fieldValue);

file:b/VOID_SaveValue.cs (new)
--- /dev/null
+++ b/VOID_SaveValue.cs
@@ -1,1 +1,115 @@
+//
+//  VOID_Config.cs
+//
+//  Author:
+//       toadicus <>
+//
+//  Copyright (c) 2013 toadicus
+//
+//  This program is free software: you can redistribute it and/or modify
+//  it under the terms of the GNU General Public License as published by
+//  the Free Software Foundation, either version 3 of the License, or
+//  (at your option) any later version.
+//
+//  This program is distributed in the hope that it will be useful,
+//  but WITHOUT ANY WARRANTY; without even the implied warranty of
+//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+//  GNU General Public License for more details.
+//
+//  You should have received a copy of the GNU General Public License
+//  along with this program.  If not, see <http://www.gnu.org/licenses/>.
+using System;
+using System.Collections.Generic;
+using KSP;
+using UnityEngine;
 
+namespace VOID
+{
+	public struct VOID_SaveValue<T> : IVOID_SaveValue
+	{
+		private T _value;
+		private Type _type;
+
+		public T value
+		{
+			get
+			{
+				return this._value;
+			}
+			set
+			{
+				this._value = value;
+			}
+		}
+
+		public Type type
+		{
+			get
+			{
+				return this._type;
+			}
+			set
+			{
+				this._type = value;
+			}
+		}
+
+		public object AsType
+		{
+			get
+			{
+				return (T)this._value;
+			}
+		}
+
+		public void SetValue(object v)
+		{
+			this._value = (T)v;
+		}
+
+		public static implicit operator T(VOID_SaveValue<T> v)
+		{
+			return v.value;
+		}
+
+		public static implicit operator VOID_SaveValue<T>(T v)
+		{
+			VOID_SaveValue<T> r = new VOID_SaveValue<T>();
+			r.value = v;
+			r.type = v.GetType();
+			if (VOID_Core.Initialized)
+			{
+				VOID_Core.Instance.configDirty = true;
+			}
+			return r;
+		}
+	}
+
+	public interface IVOID_SaveValue
+	{
+		Type type { get; }
+		object AsType { get; }
+		void SetValue(object v);
+	}
+
+	[AttributeUsage(AttributeTargets.Field)]
+	public class AVOID_SaveValue : Attribute
+	{
+		protected string _name;
+
+		public string Name
+		{
+			get
+			{
+				return this._name;
+			}
+		}
+
+		public AVOID_SaveValue(string fieldName)
+		{
+			this._name = fieldName;
+		}
+	}
+}
+
+