X-Git-Url: http://wien.tomnetworks.com/gitweb/?a=blobdiff_plain;f=mcs%2Fclass%2FSystem.Drawing%2FSystem.Drawing.Drawing2D%2FLinearGradientBrush.cs;h=d2e645d2ccbd23152f27b54043cf6ad60308cdd5;hb=177fca49175449705812fb20b6dd5b7bcbb5a901;hp=88bfcc8744f47725b982db92bedef229adf8b52a;hpb=0abc2e6270020edc4a5b4c66f93b4ae582815f20;p=mono.git diff --git a/mcs/class/System.Drawing/System.Drawing.Drawing2D/LinearGradientBrush.cs b/mcs/class/System.Drawing/System.Drawing.Drawing2D/LinearGradientBrush.cs index 88bfcc8744f..d2e645d2ccb 100644 --- a/mcs/class/System.Drawing/System.Drawing.Drawing2D/LinearGradientBrush.cs +++ b/mcs/class/System.Drawing/System.Drawing.Drawing2D/LinearGradientBrush.cs @@ -6,8 +6,7 @@ // Ravindra (rkumar@novell.com) // // Copyright (C) 2002/3 Ximian, Inc. http://www.ximian.com -// -// Copyright (C) 2004 Novell, Inc (http://www.novell.com) +// Copyright (C) 2004,2006 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 @@ -29,36 +28,38 @@ // WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. // -using System.Drawing; +using System.ComponentModel; + +namespace System.Drawing.Drawing2D { -namespace System.Drawing.Drawing2D -{ - /// - /// Summary description for LinearGradientBrush. - /// public sealed class LinearGradientBrush : Brush { RectangleF rectangle; - internal LinearGradientBrush (IntPtr native) : base (native) + internal LinearGradientBrush (IntPtr native) { + Status status = GDIPlus.GdipGetLineRect (native, out rectangle); + SetNativeBrush (native); + GDIPlus.CheckStatus (status); } public LinearGradientBrush (Point point1, Point point2, Color color1, Color color2) { + IntPtr nativeObject; Status status = GDIPlus.GdipCreateLineBrushI (ref point1, ref point2, color1.ToArgb (), color2.ToArgb (), WrapMode.Tile, out nativeObject); GDIPlus.CheckStatus (status); + SetNativeBrush (nativeObject); - Rectangle rect; - status = GDIPlus.GdipGetLineRectI (nativeObject, out rect); + status = GDIPlus.GdipGetLineRect (nativeObject, out rectangle); GDIPlus.CheckStatus (status); - rectangle = (RectangleF) rect; } public LinearGradientBrush (PointF point1, PointF point2, Color color1, Color color2) { + IntPtr nativeObject; Status status = GDIPlus.GdipCreateLineBrush (ref point1, ref point2, color1.ToArgb (), color2.ToArgb (), WrapMode.Tile, out nativeObject); GDIPlus.CheckStatus (status); + SetNativeBrush (nativeObject); status = GDIPlus.GdipGetLineRect (nativeObject, out rectangle); GDIPlus.CheckStatus (status); @@ -66,8 +67,18 @@ namespace System.Drawing.Drawing2D public LinearGradientBrush (Rectangle rect, Color color1, Color color2, LinearGradientMode linearGradientMode) { + if (linearGradientMode < LinearGradientMode.Horizontal || linearGradientMode > LinearGradientMode.BackwardDiagonal) { + throw new InvalidEnumArgumentException (nameof (linearGradientMode), unchecked ((int)linearGradientMode), typeof (LinearGradientMode)); + } + + if (rect.Width == 0 || rect.Height == 0) { + throw new ArgumentException( string.Format ("Rectangle '{0}' cannot have a width or height equal to 0.", rect.ToString ())); + } + + IntPtr nativeObject; Status status = GDIPlus.GdipCreateLineBrushFromRectI (ref rect, color1.ToArgb (), color2.ToArgb (), linearGradientMode, WrapMode.Tile, out nativeObject); GDIPlus.CheckStatus (status); + SetNativeBrush (nativeObject); rectangle = (RectangleF) rect; } @@ -78,8 +89,18 @@ namespace System.Drawing.Drawing2D public LinearGradientBrush (RectangleF rect, Color color1, Color color2, LinearGradientMode linearGradientMode) { + if (linearGradientMode < LinearGradientMode.Horizontal || linearGradientMode > LinearGradientMode.BackwardDiagonal) { + throw new InvalidEnumArgumentException (nameof (linearGradientMode), unchecked ((int)linearGradientMode), typeof (LinearGradientMode)); + } + + if (rect.Width == 0.0 || rect.Height == 0.0) { + throw new ArgumentException (string.Format ("Rectangle '{0}' cannot have a width or height equal to 0.", rect.ToString ())); + } + + IntPtr nativeObject; Status status = GDIPlus.GdipCreateLineBrushFromRect (ref rect, color1.ToArgb (), color2.ToArgb (), linearGradientMode, WrapMode.Tile, out nativeObject); GDIPlus.CheckStatus (status); + SetNativeBrush (nativeObject); rectangle = rect; } @@ -90,16 +111,28 @@ namespace System.Drawing.Drawing2D public LinearGradientBrush (Rectangle rect, Color color1, Color color2, float angle, bool isAngleScaleable) { + if (rect.Width == 0 || rect.Height == 0) { + throw new ArgumentException (string.Format ("Rectangle '{0}' cannot have a width or height equal to 0.", rect.ToString ())); + } + + IntPtr nativeObject; Status status = GDIPlus.GdipCreateLineBrushFromRectWithAngleI (ref rect, color1.ToArgb (), color2.ToArgb (), angle, isAngleScaleable, WrapMode.Tile, out nativeObject); GDIPlus.CheckStatus (status); + SetNativeBrush (nativeObject); rectangle = (RectangleF) rect; } public LinearGradientBrush (RectangleF rect, Color color1, Color color2, float angle, bool isAngleScaleable) { + if (rect.Width == 0 || rect.Height == 0) { + throw new ArgumentException (string.Format ("Rectangle '{0}' cannot have a width or height equal to 0.", rect.ToString ())); + } + + IntPtr nativeObject; Status status = GDIPlus.GdipCreateLineBrushFromRectWithAngle (ref rect, color1.ToArgb (), color2.ToArgb (), angle, isAngleScaleable, WrapMode.Tile, out nativeObject); GDIPlus.CheckStatus (status); + SetNativeBrush (nativeObject); rectangle = rect; } @@ -109,11 +142,11 @@ namespace System.Drawing.Drawing2D public Blend Blend { get { int count; - Status status = GDIPlus.GdipGetLineBlendCount (nativeObject, out count); + Status status = GDIPlus.GdipGetLineBlendCount (NativeBrush, out count); GDIPlus.CheckStatus (status); float [] factors = new float [count]; float [] positions = new float [count]; - status = GDIPlus.GdipGetLineBlend (nativeObject, factors, positions, count); + status = GDIPlus.GdipGetLineBlend (NativeBrush, factors, positions, count); GDIPlus.CheckStatus (status); Blend blend = new Blend (); @@ -123,6 +156,7 @@ namespace System.Drawing.Drawing2D return blend; } set { + // no null check, MS throws a NullReferenceException here int count; float [] factors = value.Factors; float [] positions = value.Positions; @@ -140,20 +174,21 @@ namespace System.Drawing.Drawing2D if (positions [count - 1] != 1.0F) throw new ArgumentException ("Invalid Blend object. The positions array must have 1.0 as its last element."); - Status status = GDIPlus.GdipSetLineBlend (nativeObject, factors, positions, count); + Status status = GDIPlus.GdipSetLineBlend (NativeBrush, factors, positions, count); GDIPlus.CheckStatus (status); } } + [MonoTODO ("The GammaCorrection value is ignored when using libgdiplus.")] public bool GammaCorrection { get { bool gammaCorrection; - Status status = GDIPlus.GdipGetLineGammaCorrection (nativeObject, out gammaCorrection); + Status status = GDIPlus.GdipGetLineGammaCorrection (NativeBrush, out gammaCorrection); GDIPlus.CheckStatus (status); return gammaCorrection; } set { - Status status = GDIPlus.GdipSetLineGammaCorrection (nativeObject, value); + Status status = GDIPlus.GdipSetLineGammaCorrection (NativeBrush, value); GDIPlus.CheckStatus (status); } } @@ -161,11 +196,11 @@ namespace System.Drawing.Drawing2D public ColorBlend InterpolationColors { get { int count; - Status status = GDIPlus.GdipGetLinePresetBlendCount (nativeObject, out count); + Status status = GDIPlus.GdipGetLinePresetBlendCount (NativeBrush, out count); GDIPlus.CheckStatus (status); int [] intcolors = new int [count]; float [] positions = new float [count]; - status = GDIPlus.GdipGetLinePresetBlend (nativeObject, intcolors, positions, count); + status = GDIPlus.GdipGetLinePresetBlend (NativeBrush, intcolors, positions, count); GDIPlus.CheckStatus (status); ColorBlend interpolationColors = new ColorBlend (); @@ -178,6 +213,8 @@ namespace System.Drawing.Drawing2D return interpolationColors; } set { + if (value == null) + throw new ArgumentException ("InterpolationColors is null"); int count; Color [] colors = value.Colors; float [] positions = value.Positions; @@ -199,7 +236,7 @@ namespace System.Drawing.Drawing2D for (int i = 0; i < colors.Length; i++) blend [i] = colors [i].ToArgb (); - Status status = GDIPlus.GdipSetLinePresetBlend (nativeObject, blend, positions, count); + Status status = GDIPlus.GdipSetLinePresetBlend (NativeBrush, blend, positions, count); GDIPlus.CheckStatus (status); } } @@ -207,7 +244,7 @@ namespace System.Drawing.Drawing2D public Color [] LinearColors { get { int [] colors = new int [2]; - Status status = GDIPlus.GdipGetLineColors (nativeObject, colors); + Status status = GDIPlus.GdipGetLineColors (NativeBrush, colors); GDIPlus.CheckStatus (status); Color [] linearColors = new Color [2]; linearColors [0] = Color.FromArgb (colors [0]); @@ -216,7 +253,8 @@ namespace System.Drawing.Drawing2D return linearColors; } set { - Status status = GDIPlus.GdipSetLineColors (nativeObject, value [0].ToArgb (), value [1].ToArgb ()); + // no null check, MS throws a NullReferenceException here + Status status = GDIPlus.GdipSetLineColors (NativeBrush, value [0].ToArgb (), value [1].ToArgb ()); GDIPlus.CheckStatus (status); } } @@ -230,13 +268,16 @@ namespace System.Drawing.Drawing2D public Matrix Transform { get { Matrix matrix = new Matrix (); - Status status = GDIPlus.GdipGetLineTransform (nativeObject, matrix.nativeMatrix); + Status status = GDIPlus.GdipGetLineTransform (NativeBrush, matrix.nativeMatrix); GDIPlus.CheckStatus (status); return matrix; } set { - Status status = GDIPlus.GdipSetLineTransform (nativeObject, value.nativeMatrix); + if (value == null) + throw new ArgumentNullException ("Transform"); + + Status status = GDIPlus.GdipSetLineTransform (NativeBrush, value.nativeMatrix); GDIPlus.CheckStatus (status); } } @@ -244,13 +285,17 @@ namespace System.Drawing.Drawing2D public WrapMode WrapMode { get { WrapMode wrapMode; - Status status = GDIPlus.GdipGetLineWrapMode (nativeObject, out wrapMode); + Status status = GDIPlus.GdipGetLineWrapMode (NativeBrush, out wrapMode); GDIPlus.CheckStatus (status); return wrapMode; } set { - Status status = GDIPlus.GdipSetLineWrapMode (nativeObject, value); + // note: Clamp isn't valid (context wise) but it is checked in libgdiplus + if ((value < WrapMode.Tile) || (value > WrapMode.Clamp)) + throw new InvalidEnumArgumentException ("WrapMode"); + + Status status = GDIPlus.GdipSetLineWrapMode (NativeBrush, value); GDIPlus.CheckStatus (status); } } @@ -264,13 +309,16 @@ namespace System.Drawing.Drawing2D public void MultiplyTransform (Matrix matrix, MatrixOrder order) { - Status status = GDIPlus.GdipMultiplyLineTransform (nativeObject, matrix.nativeMatrix, order); + if (matrix == null) + throw new ArgumentNullException ("matrix"); + + Status status = GDIPlus.GdipMultiplyLineTransform (NativeBrush, matrix.nativeMatrix, order); GDIPlus.CheckStatus (status); } public void ResetTransform () { - Status status = GDIPlus.GdipResetLineTransform (nativeObject); + Status status = GDIPlus.GdipResetLineTransform (NativeBrush); GDIPlus.CheckStatus (status); } @@ -281,7 +329,7 @@ namespace System.Drawing.Drawing2D public void RotateTransform (float angle, MatrixOrder order) { - Status status = GDIPlus.GdipRotateLineTransform (nativeObject, angle, order); + Status status = GDIPlus.GdipRotateLineTransform (NativeBrush, angle, order); GDIPlus.CheckStatus (status); } @@ -292,7 +340,7 @@ namespace System.Drawing.Drawing2D public void ScaleTransform (float sx, float sy, MatrixOrder order) { - Status status = GDIPlus.GdipScaleLineTransform (nativeObject, sx, sy, order); + Status status = GDIPlus.GdipScaleLineTransform (NativeBrush, sx, sy, order); GDIPlus.CheckStatus (status); } @@ -306,7 +354,7 @@ namespace System.Drawing.Drawing2D if (focus < 0 || focus > 1 || scale < 0 || scale > 1) throw new ArgumentException ("Invalid parameter passed."); - Status status = GDIPlus.GdipSetLineLinearBlend (nativeObject, focus, scale); + Status status = GDIPlus.GdipSetLineLinearBlend (NativeBrush, focus, scale); GDIPlus.CheckStatus (status); } @@ -320,7 +368,7 @@ namespace System.Drawing.Drawing2D if (focus < 0 || focus > 1 || scale < 0 || scale > 1) throw new ArgumentException ("Invalid parameter passed."); - Status status = GDIPlus.GdipSetLineSigmaBlend (nativeObject, focus, scale); + Status status = GDIPlus.GdipSetLineSigmaBlend (NativeBrush, focus, scale); GDIPlus.CheckStatus (status); } @@ -331,18 +379,17 @@ namespace System.Drawing.Drawing2D public void TranslateTransform (float dx, float dy, MatrixOrder order) { - Status status = GDIPlus.GdipTranslateLineTransform (nativeObject, dx, dy, order); + Status status = GDIPlus.GdipTranslateLineTransform (NativeBrush, dx, dy, order); GDIPlus.CheckStatus (status); } public override object Clone () { IntPtr clonePtr; - Status status = GDIPlus.GdipCloneBrush (nativeObject, out clonePtr); + Status status = GDIPlus.GdipCloneBrush (NativeBrush, out clonePtr); GDIPlus.CheckStatus (status); - LinearGradientBrush clone = new LinearGradientBrush (clonePtr); - return clone; + return new LinearGradientBrush (clonePtr); } } }