Author: krasnov
Date: 2005-11-01 10:42:56 -0500 (Tue, 01 Nov 2005)
New Revision: 52448

Modified:
   trunk/mcs/class/System.Drawing/System.Drawing/ChangeLog
   trunk/mcs/class/System.Drawing/System.Drawing/StringFormat.jvm.cs
Log:
* StringFormat.jvm.cs: Refactoring

Modified: trunk/mcs/class/System.Drawing/System.Drawing/ChangeLog
===================================================================
--- trunk/mcs/class/System.Drawing/System.Drawing/ChangeLog     2005-11-01 
15:28:38 UTC (rev 52447)
+++ trunk/mcs/class/System.Drawing/System.Drawing/ChangeLog     2005-11-01 
15:42:56 UTC (rev 52448)
@@ -1,3 +1,7 @@
+2005-11-01 Vladimir Krasnov <[EMAIL PROTECTED]>
+
+       * StringFormat.jvm.cs: Refactoring
+
 2005-10-31 Jordi Mas i Hernandez <[EMAIL PROTECTED]> 
 
        * Image.cs: Default format for MemoryBMP should be Png

Modified: trunk/mcs/class/System.Drawing/System.Drawing/StringFormat.jvm.cs
===================================================================
--- trunk/mcs/class/System.Drawing/System.Drawing/StringFormat.jvm.cs   
2005-11-01 15:28:38 UTC (rev 52447)
+++ trunk/mcs/class/System.Drawing/System.Drawing/StringFormat.jvm.cs   
2005-11-01 15:42:56 UTC (rev 52448)
@@ -32,203 +32,193 @@
 using System;
 using System.Drawing.Text;
 
-namespace System.Drawing
-{
+namespace System.Drawing {
        /// <summary>
        /// Summary description for StringFormat.
        /// </summary>
-       public sealed class StringFormat : IDisposable, ICloneable
-       {
-               private static StringFormat genericDefault;
-               private int language = 0;
-               internal CharacterRange [] CharRanges;
-
-               //local storage
-               internal StringAlignment alignment;
-               internal StringAlignment linealignment;
-               internal HotkeyPrefix prefix;
-               internal StringFormatFlags flags;
-               internal StringDigitSubstitute subst;
-               internal StringTrimming trimming;
-               internal float firstTabOffset;
-               internal float [] tabStops;
+       public sealed class StringFormat : MarshalByRefObject, IDisposable, 
ICloneable {
                
-               public StringFormat()
-               {                                          
-                       
-               }               
                
-               public StringFormat(StringFormatFlags options, int lang)
-               {
-                               flags = options;
-                               LineAlignment =  StringAlignment.Near;
-                               Alignment =  StringAlignment.Near;              
        
-                               language = lang;
+               private CharacterRange [] _charRanges;
+               private StringAlignment _alignment;
+               private StringAlignment _lineAlignment;
+               private HotkeyPrefix _hotkeyPrefix;
+               private StringFormatFlags _flags;
+               private StringDigitSubstitute _digitSubstituteMethod;
+               private int _digitSubstituteLanguage;
+               private StringTrimming _trimming;
+               
+               private float _firstTabOffset;
+               private float [] _tabStops;
+               
+               #region Constructors
+
+               public StringFormat() : this(0, 0) {                            
           
                }
                
-               public StringFormat(StringFormatFlags options):this(options,0)
-               {
+               public StringFormat(StringFormatFlags options) : 
this(options,0) {
                }
                
-//             ~StringFormat()
-//             {       
-//                     Dispose ();
-//             }
+               public StringFormat(StringFormatFlags options, int lang) {
+                       _alignment = StringAlignment.Near;
+                       _digitSubstituteLanguage = lang;
+                       _digitSubstituteMethod = StringDigitSubstitute.User;
+                       _flags = options;
+                       _hotkeyPrefix = HotkeyPrefix.None;
+                       _lineAlignment = StringAlignment.Near;
+                       _trimming = StringTrimming.Character;
+               }
                
-               public void Dispose()
-               {       
+               public StringFormat (StringFormat source) {
+                       if (source == null)
+                               throw new ArgumentNullException("format");
+
+                       _alignment = source.LineAlignment;
+                       _digitSubstituteLanguage = 
source.DigitSubstitutionLanguage;
+                       _digitSubstituteMethod = source.DigitSubstitutionMethod;
+                       _flags = source.FormatFlags;
+                       _hotkeyPrefix = source.HotkeyPrefix;
+                       _lineAlignment = source.LineAlignment;
+                       _trimming = source.Trimming;
                }
 
 
-               public StringFormat (StringFormat source)
-               {
-                       Alignment = source.Alignment;
-                       LineAlignment = source.LineAlignment;
-                       HotkeyPrefix = source.HotkeyPrefix;
-                       subst = source.subst;
-                       flags = source.flags;
+               #endregion
+
+               #region IDisposable
+
+               public void Dispose() { 
                }
 
-               
-               public StringAlignment Alignment 
-               {
-                       get 
-                       {
-                               return alignment;
+               #endregion
+
+               #region Public properties
+
+               public StringAlignment Alignment {
+                       get {
+                               return _alignment;
                        }
 
-                       set 
-                       {
-                               alignment = value;
+                       set {
+                               _alignment = value;
                        }
                }
 
-               public StringAlignment LineAlignment 
-               {
-                       get 
-                       {
-                               return linealignment;
+               public StringAlignment LineAlignment {
+                       get {
+                               return _lineAlignment;
                        }
-                       set 
-                       {
-                               linealignment = value;
+                       set {
+                               _lineAlignment = value;
                        }
                }
 
-               public StringFormatFlags FormatFlags 
-               {
-                       get 
-                       {                               
-                               return flags;
+               public StringFormatFlags FormatFlags {
+                       get {                           
+                               return _flags;
                        }
 
-                       set 
-                       {
-                               flags = value;
+                       set {
+                               _flags = value;
                        }
                }
 
-               public HotkeyPrefix HotkeyPrefix 
-               {
-                       get 
-                       {                               
-                               return prefix;
+               public HotkeyPrefix HotkeyPrefix {
+                       get {                           
+                               return _hotkeyPrefix;
                        }
 
-                       set 
-                       {
-                               prefix = value;
+                       set {
+                               _hotkeyPrefix = value;
                        }
                }
 
-
-               public StringTrimming Trimming 
-               {
-                       get 
-                       {
-                               return trimming;
+               public StringTrimming Trimming {
+                       get {
+                               return _trimming;
                        }
 
-                       set 
-                       {
-                               trimming = value;
+                       set {
+                               _trimming = value;
                        }
                }
 
-               public static StringFormat GenericDefault 
-               {
-                       get 
-                       {
-                               return genericDefault;
+               public int DigitSubstitutionLanguage {
+                       get {
+                               return _digitSubstituteLanguage;
                        }
                }
-               
-               
-               public int DigitSubstitutionLanguage 
-               {
-                       get
-                       {
-                               return language;
+
+               public StringDigitSubstitute DigitSubstitutionMethod {
+                       get {
+                               return _digitSubstituteMethod;     
                        }
                }
 
-               
-               public static StringFormat GenericTypographic 
-               {
-                       get 
-                       {
-                               throw new NotImplementedException();
+
+               #endregion
+
+               #region static properties
+
+               public static StringFormat GenericDefault {
+                       get {
+                               StringFormat genericDefault = new 
StringFormat();
+                               return genericDefault;
                        }
                }
-
-               public StringDigitSubstitute  DigitSubstitutionMethod  
-               {
-                       get 
-                       {
-                               return subst;     
+               
+               public static StringFormat GenericTypographic {
+                       get {
+                               StringFormat genericTypographic = new 
StringFormat(
+                                       StringFormatFlags.FitBlackBox |
+                                       StringFormatFlags.LineLimit |
+                                       StringFormatFlags.NoClip, 
+                                       0 );
+                               genericTypographic.Trimming = 
StringTrimming.None;
+                               return genericTypographic;
                        }
                }
 
+               #endregion
 
-               public void SetMeasurableCharacterRanges (CharacterRange [] 
range)
-               {
-                       CharRanges=(CharacterRange [])range.Clone();
+               #region public methods
+
+               public void SetMeasurableCharacterRanges (CharacterRange [] 
range) {
+                       _charRanges = (CharacterRange [])range.Clone();
                }
 
-               internal CharacterRange [] GetCharRanges
-               {
-                       get 
-                       {
-                               return(CharRanges);
+               internal CharacterRange [] GetCharRanges {
+                       get {
+                               return _charRanges;
                        }
                }
        
-               public object Clone()
-               {
-                       throw new NotImplementedException();
+               public object Clone() {
+                       StringFormat copy = (StringFormat)MemberwiseClone();
+                       copy._charRanges = (CharacterRange 
[])_charRanges.Clone();
+                       copy._tabStops = (float[])_tabStops.Clone();
+                       return copy;
                }
 
-               public override string ToString()
-               {
+               public override string ToString() {
                        return "[StringFormat, FormatFlags=" + 
this.FormatFlags.ToString() + "]";
                }
                
-               public void SetTabStops(float firstTabOffset, float[] tabStops)
-               {
-                       this.firstTabOffset = firstTabOffset;
-                       this.tabStops = tabStops;
+               public void SetTabStops(float firstTabOffset, float[] tabStops) 
{
+                       _firstTabOffset = firstTabOffset;
+                       _tabStops = (float[])tabStops.Clone();
                }
 
-               public void SetDigitSubstitution(int language,  
StringDigitSubstitute substitute)
-               {
-                       subst = substitute;
+               public void SetDigitSubstitution(int language,  
StringDigitSubstitute substitute) {
+                       _digitSubstituteMethod = substitute;
+                       _digitSubstituteLanguage = language;
                }
 
-               public float[] GetTabStops(out float firstTabOffset)
-               {
-                       firstTabOffset = this.firstTabOffset;
-                       return this.tabStops;
+               public float[] GetTabStops(out float firstTabOffset) {
+                       firstTabOffset = _firstTabOffset;
+                       return (float[])_tabStops.Clone();
                }
 
+               #endregion
        }
 }

_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches

Reply via email to