2008-03-28 Ivan N. Zlatev <contact@i-nz.net>

[mono.git] / mcs / class / System.Drawing / System.Drawing.Printing / Margins.cs

diff --git a/mcs/class/System.Drawing/System.Drawing.Printing/Margins.cs b/mcs/class/System.Drawing/System.Drawing.Printing/Margins.cs
index f505533dde4c42ac99c1c77b00bb8eb4420ce7d0..6a4f959d9cfe614cac78a1caa5f9d34b7130993d 100644 (file)
--- a/mcs/class/System.Drawing/System.Drawing.Printing/Margins.cs
+++ b/mcs/class/System.Drawing/System.Drawing.Printing/Margins.cs
@@ -4,12 +4,10 @@
 // Authors:
 //   Dennis Hayes (dennish@Raytek.com)
 //   Andreas Nahr (ClassDevelopment@A-SoftTech.com)
+//   Sebastien Pouliot  <sebastien@ximian.com>
 //
 // (C) 2002 Ximian, Inc
-//
-
-//
-// Copyright (C) 2004 Novell, Inc (http://www.novell.com)
+// Copyright (C) 2004, 2007 Novell, Inc (http://www.novell.com)
 //
 // Permission is hereby granted, free of charge, to any person obtaining
 // a copy of this software and associated documentation files (the
@@ -31,20 +29,21 @@
 // WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 //
 
-using System;
 using System.ComponentModel;
 
 namespace System.Drawing.Printing
 {
+#if NET_2_0
+       [Serializable]
+#endif
        [TypeConverter (typeof (MarginsConverter))]
-       public class Margins : ICloneable
-       {
+       public class Margins : ICloneable {
                int left;
                int right;
                int top;
                int bottom;
 
-               public Margins()
+               public Margins ()
                {
                        left = 100;
                        right = 100;
@@ -52,23 +51,12 @@ namespace System.Drawing.Printing
                        bottom = 100;
                }
 
-               public Margins(int left, int right, int top, int bottom)
+               public Margins (int left, int right, int top, int bottom)
                {
-                       //Verify parameters
-                       if (left < 0)
-                               throw new System.ArgumentException("All Margins must be greater than 0", "left");
-                       if (right < 0)
-                               throw new System.ArgumentException("All Margins must be greater than 0", "right");
-                       if (top < 0)
-                               throw new System.ArgumentException("All Margins must be greater than 0", "top");
-                       if (bottom < 0)
-                               throw new System.ArgumentException("All Margins must be greater than 0", "bottom");
-
-                       //Set proprities
-                       this.left = left;
-                       this.right = right;
-                       this.top = top;
-                       this.bottom = bottom;
+                       Left = left;
+                       Right = right;
+                       Top = top;
+                       Bottom = bottom;
                }
 
                public int Left {
@@ -76,8 +64,8 @@ namespace System.Drawing.Printing
                                return left;
                        }
                        set {
-                               if (left < 0)
-                                       throw new System.ArgumentException("All Margins must be greater than 0", "left");
+                               if (value < 0)
+                                       InvalidMargin ("left");
                                left = value;
                        }
                }
@@ -87,8 +75,8 @@ namespace System.Drawing.Printing
                                return right;
                        }
                        set {
-                               if (right < 0)
-                                       throw new System.ArgumentException("All Margins must be greater than 0", "left");
+                               if (value < 0)
+                                       InvalidMargin ("right");
                                right = value;
                        }
                }
@@ -98,8 +86,8 @@ namespace System.Drawing.Printing
                                return top;
                        }
                        set {
-                               if (top < 0)
-                                       throw new System.ArgumentException("All Margins must be greater than 0", "left");
+                               if (value < 0)
+                                       InvalidMargin ("top");
                                top = value;
                        }
                }
@@ -109,40 +97,63 @@ namespace System.Drawing.Printing
                                return bottom;
                        }
                        set {
-                               if (bottom < 0)
-                                       throw new System.ArgumentException("All Margins must be greater than 0", "left");
+                               if (value < 0)
+                                       InvalidMargin ("bottom");
                                bottom = value;
                        }
                }
+
+               private void InvalidMargin (string property)
+               {
+                       string msg = Locale.GetText ("All Margins must be greater than 0");
+                       throw new System.ArgumentException (msg, property);
+               }
                
-               public object Clone()
+               public object Clone ()
                {
-                       return new Margins (this.Left, this.Right, this.Top, this.Bottom);
+                       return new Margins (left, right, top, bottom);
                }
 
                public override bool Equals (object obj)
                {       
-                       Margins m = obj as Margins;
+                       return Equals (obj as Margins);
+               }
 
-                       if (m == null)
+               private bool Equals (Margins m)
+               {
+                       // avoid recursion with == operator
+                       if ((object)m == null)
                                return false;
-                       if (m.Left == left && m.Right == right && m.Top == top && m.Bottom == bottom)
-                               return true;
-
-                       return false;
+                       return ((m.Left == left) && (m.Right == right) && (m.Top == top) && (m.Bottom == bottom));
                }
 
                public override int GetHashCode ()
                {
-                       // Try to create a somewhat meaningful hash
-                       int hash = left + right * 2^8 + top * 2^16 + bottom * 2^24;
-                       return hash;
+                       return left | (right << 8) | (right >> 24) | (top << 16) | (top >> 16) | (bottom << 24) | (bottom >> 8);
                }
                
-               public override string ToString()
+               public override string ToString ()
                {
                        string ret = "[Margins Left={0} Right={1} Top={2} Bottom={3}]";
-                       return String.Format (ret, this.Left, this.Right, this.Top, this.Bottom);
+                       return String.Format (ret, left, right, top, bottom);
+               }
+
+#if NET_2_0
+               public static bool operator == (Margins m1, Margins m2)
+               {
+                       // avoid recursion with == operator
+                       if ((object)m1 == null)
+                               return ((object)m2 == null);
+                       return m1.Equals (m2);
+               }
+
+               public static bool operator != (Margins m1, Margins m2)
+               {
+                       // avoid recursion with == operator
+                       if ((object)m1 == null)
+                               return ((object)m2 != null);
+                       return !m1.Equals (m2);
                }
+#endif
        }
 }