2006-12-01 Sebastien Pouliot <sebastien@ximian.com>

	* SmtpClient.cs: Added MonoTODO for missing SSL/TLS support and
	updated existing MonoTODO (or changed them to FIXME) to be clearer
	for anyone using the API.

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

diff --git a/mcs/class/System/System.Net.Mail/ChangeLog b/mcs/class/System/System.Net.Mail/ChangeLog
index 38cc894bb9561bc90f9790a78a26b9e01e41c8ee..efc7403b970903c2b6e9ecbe892c9f24c37da08a 100644 (file)
--- a/mcs/class/System/System.Net.Mail/ChangeLog
+++ b/mcs/class/System/System.Net.Mail/ChangeLog
@@ -1,3 +1,9 @@
+2006-12-01  Sebastien Pouliot  <sebastien@ximian.com>
+
+       * SmtpClient.cs: Added MonoTODO for missing SSL/TLS support and 
+       updated existing MonoTODO (or changed them to FIXME) to be clearer
+       for anyone using the API.
+
 2006-10-31  Sebastien Pouliot  <sebastien@ximian.com>
 
        * SmtpException.cs: Fix visibility on .ctor(SerializationInfo,

diff --git a/mcs/class/System/System.Net.Mail/SmtpClient.cs b/mcs/class/System/System.Net.Mail/SmtpClient.cs
index 15fb60073a27f10f41714e14f2b82fff97bb9f43..7bb5335e00f0dada39fd8839e4bd255b96296a83 100644 (file)
--- a/mcs/class/System/System.Net.Mail/SmtpClient.cs
+++ b/mcs/class/System/System.Net.Mail/SmtpClient.cs
@@ -80,7 +80,7 @@ namespace System.Net.Mail {
                {
                }
 
-               [MonoTODO ("Load default settings from configuration.")]
+               [MonoTODO ("Default settings aren't loaded from configuration files.")]
                public SmtpClient (string host, int port)
                {
                        // FIXME: load from configuration
@@ -102,7 +102,7 @@ namespace System.Net.Mail {
 
                #region Properties
 
-               [MonoTODO]
+               [MonoTODO ("SSL/TLS support isn't available.")]
                public X509CertificateCollection ClientCertificates {
                        get { return clientCertificates; }
                }
@@ -117,6 +117,7 @@ namespace System.Net.Mail {
                        set { deliveryMethod = value; }
                }
 
+               [MonoTODO ("SSL/TLS support isn't available.")]
                public bool EnableSsl {
                        get { return enableSsl; }
                        set { enableSsl = value; }
@@ -124,7 +125,7 @@ namespace System.Net.Mail {
 
                public string Host {
                        get { return host; }
-                       [MonoTODO ("Check to make sure an email is not being sent.")]
+                       // FIXME: Check to make sure an email is not being sent.
                        set {
                                if (value == null)
                                        throw new ArgumentNullException ();
@@ -141,7 +142,7 @@ namespace System.Net.Mail {
 
                public int Port {
                        get { return port; }
-                       [MonoTODO ("Check to make sure an email is not being sent.")]
+                       // FIXME: Check to make sure an email is not being sent.
                        set { 
                                if (value <= 0)
                                        throw new ArgumentOutOfRangeException ();
@@ -156,7 +157,7 @@ namespace System.Net.Mail {
 
                public int Timeout {
                        get { return timeout; }
-                       [MonoTODO ("Check to make sure an email is not being sent.")]
+                       // FIXME: Check to make sure an email is not being sent.
                        set { 
                                if (value < 0)
                                        throw new ArgumentOutOfRangeException ();
@@ -164,7 +165,7 @@ namespace System.Net.Mail {
                        }
                }
 
-               [MonoTODO]
+               [MonoTODO ("This property is current ignored.")]
                public bool UseDefaultCredentials {
                        get { return useDefaultCredentials; }
                        set { useDefaultCredentials = value; }
@@ -222,7 +223,7 @@ namespace System.Net.Mail {
                        return response;
                }
 
-               [MonoTODO ("Need to work on message attachments.")]
+               [MonoTODO ("Message attachments support is incomplete.")]
                public void Send (MailMessage message)
                {
                        // Block while sending