Allow configuration settings to be removed. Previously, if you tried to

remove a configuration setting, you'd get an exception saying that there was
no factory for it (i.e. it had already been removed).

svn path=/trunk/mcs/; revision=9664

diff --git a/mcs/class/System/System.Configuration/ChangeLog b/mcs/class/System/System.Configuration/ChangeLog
index 24db59f9e8dc670ef016d1db698d26206dab36fd..c136e57ca4d30a333cbdf65d3f9bc2b4ffa88d0d 100644 (file)
--- a/mcs/class/System/System.Configuration/ChangeLog
+++ b/mcs/class/System/System.Configuration/ChangeLog
@@ -1,3 +1,7 @@
+2002-12-14  Jonathan Pryor <jonpryor@vt.edu>
+
+       * ConfigurationSettings.cs: Allow configuration sections to be removed.
+
 2002-11-04  Gonzalo Paniagua Javier <gonzalo@ximian.com>
 
        * ConfigurationSettings.cs: use AppDomainSetup to get the configuration

diff --git a/mcs/class/System/System.Configuration/ConfigurationSettings.cs b/mcs/class/System/System.Configuration/ConfigurationSettings.cs
index 5f519b0060ca0c16d035406eab52add7b7c3ed42..42334d7c50431d1718ab383ebec20445ba5ee386 100644 (file)
--- a/mcs/class/System/System.Configuration/ConfigurationSettings.cs
+++ b/mcs/class/System/System.Configuration/ConfigurationSettings.cs
@@ -374,7 +374,7 @@ namespace System.Configuration
                                removeValue = sectionName + '/' + removeValue;
 
                        object o = LookForFactory (removeValue);
-                       if (o != null && o != removedMark)
+                       if (o != null && o == removedMark)
                                ThrowException ("No factory for " + removeValue, reader);
 
                        factories [removeValue] = removedMark;