Author: kostat
Date: 2006-11-16 03:28:11 -0500 (Thu, 16 Nov 2006)
New Revision: 67974

Modified:
   trunk/mcs/class/System.Configuration/System.Configuration/ChangeLog
   trunk/mcs/class/System.Configuration/System.Configuration/SectionInfo.cs
Log:
refactoring - remove unneeded bool? usage

Modified: trunk/mcs/class/System.Configuration/System.Configuration/ChangeLog
===================================================================
--- trunk/mcs/class/System.Configuration/System.Configuration/ChangeLog 
2006-11-16 05:49:19 UTC (rev 67973)
+++ trunk/mcs/class/System.Configuration/System.Configuration/ChangeLog 
2006-11-16 08:28:11 UTC (rev 67974)
@@ -1,3 +1,7 @@
+2006-08-23  Konstantin Triger <[EMAIL PROTECTED]>
+
+       * SectionInfo.cs: refactoring - remove unneeded bool? usage.
+       
 2006-11-05  Vladimir Krasnov  <[EMAIL PROTECTED]>
 
        * added: ConfigurationRemoveElement.cs, common element for <remove>

Modified: 
trunk/mcs/class/System.Configuration/System.Configuration/SectionInfo.cs
===================================================================
--- trunk/mcs/class/System.Configuration/System.Configuration/SectionInfo.cs    
2006-11-16 05:49:19 UTC (rev 67973)
+++ trunk/mcs/class/System.Configuration/System.Configuration/SectionInfo.cs    
2006-11-16 08:28:11 UTC (rev 67974)
@@ -38,7 +38,7 @@
        internal class SectionInfo: ConfigInfo
        {
                bool allowLocation = true;
-               bool? requirePermission = true;
+               bool requirePermission = true;
                ConfigurationAllowDefinition allowDefinition = 
ConfigurationAllowDefinition.Everywhere;
                ConfigurationAllowExeDefinition allowExeDefinition = 
ConfigurationAllowExeDefinition.MachineToApplication;
 
@@ -64,8 +64,7 @@
                                sec.SectionInformation.AllowLocation = 
allowLocation;
                                sec.SectionInformation.AllowDefinition = 
allowDefinition;
                                sec.SectionInformation.AllowExeDefinition = 
allowExeDefinition;
-                               if (requirePermission != null)
-                                       
sec.SectionInformation.RequirePermission = requirePermission.Value;
+                               sec.SectionInformation.RequirePermission = 
requirePermission;
                                sec.SectionInformation.SetName (Name);
                        }
                        return ob;
@@ -158,8 +157,8 @@
                                writer.WriteAttributeString ("allowDefinition", 
allowDefinition.ToString ());
                        if (allowExeDefinition != 
ConfigurationAllowExeDefinition.MachineToApplication)
                                writer.WriteAttributeString 
("allowExeDefinition", allowExeDefinition.ToString ());
-                       if (requirePermission != null)
-                               writer.WriteAttributeString 
("requirePermission", requirePermission.Value ? "true" : "false");
+                       if (!requirePermission)
+                               writer.WriteAttributeString 
("requirePermission", "false");
                        writer.WriteEndElement ();
                }
                

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

Reply via email to