Merge pull request #1030 from ivandrofly/patch-27

Prefer item.SubItems.Add()
This commit is contained in:
Nikolaj Olsson 2015-07-22 20:56:02 +02:00
commit bc0776c70b

View File

@ -483,12 +483,8 @@ namespace Nikse.SubtitleEdit.Forms
private void AddFixActionItemToListView(FixItem fi)
{
var item = new ListViewItem(string.Empty) { Tag = fi, Checked = fi.DefaultChecked };
var subItem = new ListViewItem.ListViewSubItem(item, fi.Name);
item.SubItems.Add(subItem);
subItem = new ListViewItem.ListViewSubItem(item, fi.Example);
item.SubItems.Add(subItem);
item.SubItems.Add(fi.Name);
item.SubItems.Add(fi.Example);
listView1.Items.Add(item);
}
@ -496,19 +492,11 @@ namespace Nikse.SubtitleEdit.Forms
{
if (_onlyListFixes)
{
var item = new ListViewItem(string.Empty) { Checked = true };
var subItem = new ListViewItem.ListViewSubItem(item, p.Number.ToString(CultureInfo.InvariantCulture));
item.SubItems.Add(subItem);
subItem = new ListViewItem.ListViewSubItem(item, action);
item.SubItems.Add(subItem);
subItem = new ListViewItem.ListViewSubItem(item, before.Replace(Environment.NewLine, Configuration.Settings.General.ListViewLineSeparatorString));
item.SubItems.Add(subItem);
subItem = new ListViewItem.ListViewSubItem(item, after.Replace(Environment.NewLine, Configuration.Settings.General.ListViewLineSeparatorString));
item.SubItems.Add(subItem);
item.Tag = p; // save paragraph in Tag
var item = new ListViewItem(string.Empty) { Checked = true, Tag = p };
item.SubItems.Add(p.Number.ToString(CultureInfo.InvariantCulture));
item.SubItems.Add(action);
item.SubItems.Add(before.Replace(Environment.NewLine, Configuration.Settings.General.ListViewLineSeparatorString));
item.SubItems.Add(after.Replace(Environment.NewLine, Configuration.Settings.General.ListViewLineSeparatorString));
listViewFixes.Items.Add(item);
}
}