2008-12-17 Bill Holmes <billholmes54@gmail.com>

	* ListViewItem.cs (ListViewSubItem.ctor): Initalizing the
	  SubItemStyle member field.

	Contributed under MIT/X11 license.

2008-12-17  Jonathan Pobst  <monkey@jpobst.com>

	* ListViewItemTest.cs: Add test for empty constructor.

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

diff --git a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog
index ed3665de7784331471353707d9c8f094a9bf0e20..ddba82f3d5a47099cf981d4dd36789f18e7f1e7a 100644 (file)
--- a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog
+++ b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog
@@ -1,3 +1,10 @@
+2008-12-17  Bill Holmes  <billholmes54@gmail.com>
+
+       * ListViewItem.cs (ListViewSubItem.ctor): Initalizing the 
+         SubItemStyle member field. 
+
+       Contributed under MIT/X11 license.
+
 2008-12-16  Carlos Alberto Cortez <calberto.cortez@gmail.com>
 
        * ListBox.cs: In MultiColumn mode don't use top_index to calculate the

diff --git a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListViewItem.cs b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListViewItem.cs
index e3a01ab1b5e5b0eb361ee1bff2ca98953119a2f2..b54f263580ca0a6d7eb5223b82010ed2c18b26eb 100644 (file)
--- a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListViewItem.cs
+++ b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListViewItem.cs
@@ -1223,6 +1223,8 @@ namespace System.Windows.Forms
                        
                        #region Public Constructors
                        public ListViewSubItem ()
+                               : this (null, string.Empty, Color.Empty,
+                                       Color.Empty, null)
                        {
                        }
 

diff --git a/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ChangeLog b/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ChangeLog
index 93d9138ec54ad2e5a13c65c3f85f9a93c765a48b..286d3b98e9baae6e2c427a93c18e0f398acb166d 100644 (file)
--- a/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ChangeLog
+++ b/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ChangeLog
@@ -1,3 +1,7 @@
+2008-12-17  Jonathan Pobst  <monkey@jpobst.com>
+
+       * ListViewItemTest.cs: Add test for empty constructor.
+
 2008-12-15  Mario Carrion <mcarrion@novell.com>
 
        * ControlTest.cs: New test added: AccessibilityPropertiesTest to test

diff --git a/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ListViewItemTest.cs b/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ListViewItemTest.cs
index 5c03d5753cf1776cac7130ba8eda158542157672..113e969776b5b737e244b46435896be10d58ac75 100644 (file)
--- a/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ListViewItemTest.cs
+++ b/mcs/class/Managed.Windows.Forms/Test/System.Windows.Forms/ListViewItemTest.cs
@@ -701,6 +701,9 @@ namespace MonoTests.System.Windows.Forms
                {
                        ListViewItem.ListViewSubItem subItem = new ListViewItem.ListViewSubItem ();
                        Assert.AreEqual (string.Empty, subItem.Text, "#1");
+                       Assert.AreEqual (SystemColors.Window, subItem.BackColor, "#2");
+                       Assert.AreEqual (SystemColors.WindowText, subItem.ForeColor, "#3");
+                       Assert.AreEqual (SystemFonts.DefaultFont, subItem.Font, "#4");
                }
 
                [Test] // ctor (ListViewItem, String)