proposed fix for issue #24 when cloning objects implementing INotityP…

DeepCloner/Helpers/DeepClonerExprGenerator.cs

@@ -141,8 +141,10 @@ private static object GenerateProcessMethod(Type type, bool unboxStruct)
 				if (tp.Name == "ContextBoundObject") break;
 #endif
 
-				fi.AddRange(tp.GetDeclaredFields());
-				tp = tp.BaseType();
+                // Avoid to copy property change event handlers (or should we avoid to copy event handlers at all?)
+                var fields = tp.GetDeclaredFields().Where(x => !x.IsPropertyChangeEventHandler());
+                fi.AddRange(fields);
+                tp = tp.BaseType();
 			}
 			while (tp != null);
 

DeepCloner/Helpers/DeepClonerGenerator.cs

@@ -1,4 +1,5 @@
 using System;
+using System.ComponentModel;
 using System.Linq;
 
 namespace Force.DeepCloner.Helpers
@@ -33,8 +34,10 @@ private static object CloneClassRoot(object obj)
 			if (cloner == null) 
 				return obj;
 
-			return cloner(obj, new DeepCloneState());
-		}
+            var result = cloner(obj, new DeepCloneState());
+            result = RemovePropertyChangeEventHandler(result);
+            return result;
+        }
 
 		internal static object CloneClassInternal(object obj, DeepCloneState state)
 		{
@@ -52,10 +55,29 @@ internal static object CloneClassInternal(object obj, DeepCloneState state)
 			if (knownRef != null) 
 				return knownRef;
 
-			return cloner(obj, state);
-		}
-
-		private static T CloneStructInternal<T>(T obj, DeepCloneState state) // where T : struct
+            var result = cloner(obj, state);
+            result = RemovePropertyChangeEventHandler(result);
+            return result;
+        }
+
+        /// <summary>
+        /// Clears fields that store <see cref="PropertyChangingEventHandler"/> or <see cref="PropertyChangedEventHandler"/> instances.
+        /// </summary>
+        private static object RemovePropertyChangeEventHandler(object o)
+        {
+            var t = o.GetType();
+            var fields = o.GetType().GetAllFieldsIncludingBaseTypes();
+            foreach (var fieldInfo in fields)
+            {
+                if (fieldInfo.IsPropertyChangeEventHandler())
+                {
+                    fieldInfo.SetValue(o, null);
+                }
+            }
+            return o;
+        }
+
+        private static T CloneStructInternal<T>(T obj, DeepCloneState state) // where T : struct
 		{
 			// no loops, no nulls, no inheritance
 			var cloner = GetClonerForValueType<T>();

DeepCloner/Helpers/ReflectionHelper.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Collections.Generic;
+using System.ComponentModel;
 using System.Linq;
 using System.Reflection;
 
@@ -50,8 +52,33 @@ public static FieldInfo[] GetAllFields(this Type t)
 			return t.GetFields(BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public);
 #endif
 		}
-
-		public static PropertyInfo[] GetPublicProperties(this Type t)
+
+        /// <summary>
+        /// Returns all fields declared on the specified type and its base types.
+        /// </summary>
+        /// <param name="t"></param>
+        internal static FieldInfo[] GetAllFieldsIncludingBaseTypes(this Type t)
+        {
+            var fields = new List<FieldInfo>();
+            Type currentType = t;
+            do
+            {
+                fields.AddRange(currentType.GetAllFields());
+                currentType = currentType.BaseType();
+            }
+            while (currentType != null);
+            return fields.ToArray();
+        }
+
+        /// <summary>
+        /// Indicates whether the field type is <see cref="PropertyChangingEventHandler"/> or <see cref="PropertyChangedEventHandler"/>.
+        /// </summary>
+        internal static bool IsPropertyChangeEventHandler(this FieldInfo f)
+        {
+            return f.FieldType == typeof(PropertyChangingEventHandler) || f.FieldType == typeof(PropertyChangedEventHandler);
+        }
+
+        public static PropertyInfo[] GetPublicProperties(this Type t)
 		{
 #if NETCORE
 			return t.GetTypeInfo().DeclaredProperties.ToArray();