Simplified the conversions, more already available TypeConverters are…

… now taken.

Dapplo.Config.Test/IniTest.cs

@@ -68,6 +68,8 @@ public async Task TestIniInit()
 			}
 			var iniTest = await iniConfig.RegisterAndGetAsync<IIniTest>().ConfigureAwait(false);
 			Assert.IsTrue(iniTest.Height == 185);
+			Assert.IsTrue(iniTest.PropertySize.Width == 16);
+			Assert.IsTrue(iniTest.PropertyArea.Width == 100);
 			Assert.IsTrue(iniTest.WindowCornerCutShape.Count > 0);
 			Assert.IsTrue(iniTest.SomeValues.ContainsKey("dapplo"));
 

Dapplo.Config.Test/TestInterfaces/IIniTest.cs

@@ -23,6 +23,7 @@
 using Dapplo.Config.Ini;
 using System.Collections.Generic;
 using System.ComponentModel;
+using System.Drawing;
 using System.Runtime.Serialization;
 
 namespace Dapplo.Config.Test.TestInterfaces
@@ -81,5 +82,16 @@ IDictionary<string, int> SomeValues
 			get;
 			set;
 		}
+
+		[DefaultValue("16,16")]
+		Size PropertySize {
+			get;
+			set;
+		}
+		[DefaultValue("16,16,100,100")]
+		Rectangle PropertyArea {
+			get;
+			set;
+		}
 	}
 }

Dapplo.Config/Dapplo.Config.csproj

@@ -37,7 +37,6 @@
     <Reference Include="WindowsBase" />
   </ItemGroup>
   <ItemGroup>
-    <Compile Include="Converters\SystemDrawingTypeConverters.cs" />
     <Compile Include="DapploConfig.cs" />
     <Compile Include="ExtensionAttribute.cs" />
     <Compile Include="Extension\IDescription.cs" />

Dapplo.Config/Extension/Implementation/DefaultValueExtension.cs

@@ -76,7 +76,12 @@ private object GetConvertedDefaultValue(PropertyInfo propertyInfo)
 				if (typeConverter != null && typeConverter.CanConvertFrom(defaultValue.GetType()))
 				{
 					// Convert
-					return typeConverter.ConvertFrom(defaultValue);
+					var defaultStringValue = defaultValue as string;
+					if (defaultStringValue != null) {
+						return typeConverter.ConvertFromInvariantString(defaultStringValue);
+					} else {
+						return typeConverter.ConvertFrom(defaultValue);
+					}
 				}
 			}
 			return defaultValue;

Dapplo.Config/Ini/IniConfig.cs

@@ -117,19 +117,14 @@ public static IniConfig Get(string applicationName, string fileName)
 		/// <param name="fileName"></param>
 		/// <param name="fixedDirectory">Specify a path if you don't want to use the default loading</param>
 		/// <param name="registerDefaultConverters">false if you don't want to have any default converters</param>
-		public IniConfig(string applicationName, string fileName, string fixedDirectory = null, bool registerDefaultConverters = true)
+		public IniConfig(string applicationName, string fileName, string fixedDirectory = null)
 		{
 			_applicationName = applicationName;
 			_fileName = fileName;
 			_fixedDirectory = fixedDirectory;
 			// Look for the ini file, this is only done 1 time.
 			_iniFile = CreateFileLocation(false, "", _fixedDirectory);
 
-			if (registerDefaultConverters)
-			{
-				SetDefaultConverters();
-			}
-
 			WriteErrorHandler = (iniSection, iniValue, exception) =>
 			{
 				if (!iniValue.Behavior.IgnoreErrors)
@@ -159,20 +154,6 @@ public IniConfig SetDefaultConverter(Type type, Type typeConverter)
 			return this;
 		}
 
-		/// <summary>
-		/// Set the default converters
-		/// </summary>
-		public IniConfig SetDefaultConverters()
-		{
-			// System.Drawing
-			SetDefaultConverter(typeof(System.Drawing.Size), typeof(SizeTypeConverter));
-			SetDefaultConverter(typeof(System.Drawing.Point), typeof(PointTypeConverter));
-			SetDefaultConverter(typeof(System.Drawing.Rectangle), typeof(RectangleTypeConverter));
-			SetDefaultConverter(typeof(System.Drawing.Color), typeof(ColorTypeConverter));
-
-			return this;
-		}
-
 		/// <summary>
 		/// Set the after load action for a IIniSection
 		/// </summary>