From bdc2cfdf1d3080657ec15b77cfdb5870bd9c7cdb Mon Sep 17 00:00:00 2001
From: Michael Hutchinson <m.j.hutchinson@gmail.com>
Date: Wed, 6 Mar 2013 00:35:46 -0500
Subject: [PATCH] cairo: Clean up Region's Dispose/ctor

---
 cairo/Region.cs | 32 ++++++++++++++++++++++----------
 1 file changed, 22 insertions(+), 10 deletions(-)

diff --git a/cairo/Region.cs b/cairo/Region.cs
index 01a4523b3..465d0ad6a 100644
--- a/cairo/Region.cs
+++ b/cairo/Region.cs
@@ -47,11 +47,7 @@ namespace Cairo
 			get { return handle; }
 		}
 
-		~Region ()
-		{
-			Console.WriteLine ("Cairo.Region finalizer reached - developer must dispose regions manually to avoid leakage due to thread-safety concerns.");
-		}
-
+		[Obsolete]
 		public Region (IntPtr handle) : this (handle, false) {}
 
 		public Region (IntPtr handle, bool owned)
@@ -59,11 +55,12 @@ namespace Cairo
 			this.handle = handle;
 			if (!owned)
 				NativeMethods.cairo_region_reference (handle);
+			if (CairoDebug.Enabled)
+				CairoDebug.OnAllocated (handle);
 		}
 
-		public Region ()
+		public Region () : this (NativeMethods.cairo_region_create () , true)
 		{
-			handle = NativeMethods.cairo_region_create ();
 		}
 
 		public Region (RectangleInt rect)
@@ -81,14 +78,29 @@ namespace Cairo
 			return new Region (NativeMethods.cairo_region_copy (Handle), true);
 		}
 
+		~Region ()
+		{
+			Dispose (false);
+		}
+
 		public void Dispose ()
 		{
-			if (handle != IntPtr.Zero)
-				NativeMethods.cairo_region_destroy (Handle);
-			handle = IntPtr.Zero;
+			Dispose (true);
 			GC.SuppressFinalize (this);
 		}
 
+		protected virtual void Dispose (bool disposing)
+		{
+			if (!disposing || CairoDebug.Enabled)
+				CairoDebug.OnDisposed<Region> (handle, disposing);
+
+			if (!disposing|| handle == IntPtr.Zero)
+				return;
+
+			NativeMethods.cairo_region_destroy (Handle);
+			handle = IntPtr.Zero;
+		}
+
 		public override bool Equals (object obj)
 		{
 			return (obj is Region) && NativeMethods.cairo_region_equal (Handle, (obj as Region).Handle);