From 6d8b6f79b44b6a41a630aa3aad5a3c7f16701798 Mon Sep 17 00:00:00 2001 From: Sean Hall Date: Sat, 11 Apr 2020 21:49:09 +1000 Subject: General cleanup. Try not to send strings to specify the tuple or table. Try to avoid using the Set method on tuples. Always create new tuples and add them to the section in the same line. --- .../Bind/AssignMediaCommand.cs | 31 ++++++++++------------ 1 file changed, 14 insertions(+), 17 deletions(-) (limited to 'src/WixToolset.Core.WindowsInstaller/Bind/AssignMediaCommand.cs') diff --git a/src/WixToolset.Core.WindowsInstaller/Bind/AssignMediaCommand.cs b/src/WixToolset.Core.WindowsInstaller/Bind/AssignMediaCommand.cs index 2bfd587f..ae7e5788 100644 --- a/src/WixToolset.Core.WindowsInstaller/Bind/AssignMediaCommand.cs +++ b/src/WixToolset.Core.WindowsInstaller/Bind/AssignMediaCommand.cs @@ -71,12 +71,12 @@ namespace WixToolset.Core.WindowsInstaller.Bind // When building merge module, all the files go to "#MergeModule.CABinet". if (SectionType.Module == this.Section.Type) { - var mergeModuleMediaRow = new MediaTuple(); - mergeModuleMediaRow.Cabinet = "#MergeModule.CABinet"; - - this.Section.Tuples.Add(mergeModuleMediaRow); + var mergeModuleMediaTuple = this.Section.AddTuple(new MediaTuple + { + Cabinet = "#MergeModule.CABinet", + }); - filesByCabinetMedia.Add(mergeModuleMediaRow, new List(this.FileFacades)); + filesByCabinetMedia.Add(mergeModuleMediaTuple, new List(this.FileFacades)); } else if (mediaTemplateTable.Count == 0) { @@ -212,13 +212,12 @@ namespace WixToolset.Core.WindowsInstaller.Bind // If there are uncompressed files and no MediaRow, create a default one. if (uncompressedFiles.Count > 0 && !this.Section.Tuples.OfType().Any()) { - var defaultMediaRow = new MediaTuple(null, new Identifier(AccessModifier.Private, 1)) + var defaultMediaRow = this.Section.AddTuple(new MediaTuple(null, new Identifier(AccessModifier.Private, 1)) { - DiskId = 1 - }; + DiskId = 1, + }); mediaRows.Add(1, defaultMediaRow); - this.Section.Tuples.Add(defaultMediaRow); } } @@ -298,14 +297,12 @@ namespace WixToolset.Core.WindowsInstaller.Bind /// private MediaTuple AddMediaRow(WixMediaTemplateTuple mediaTemplateTuple, int cabIndex) { - var currentMediaTuple = new MediaTuple(mediaTemplateTuple.SourceLineNumbers, new Identifier(AccessModifier.Private, cabIndex)); - currentMediaTuple.DiskId = cabIndex; - currentMediaTuple.Cabinet = String.Format(CultureInfo.InvariantCulture, this.CabinetNameTemplate, cabIndex); - currentMediaTuple.CompressionLevel = mediaTemplateTuple.CompressionLevel; - - this.Section.Tuples.Add(currentMediaTuple); - - return currentMediaTuple; + return this.Section.AddTuple(new MediaTuple(mediaTemplateTuple.SourceLineNumbers, new Identifier(AccessModifier.Private, cabIndex)) + { + DiskId = cabIndex, + Cabinet = String.Format(CultureInfo.InvariantCulture, this.CabinetNameTemplate, cabIndex), + CompressionLevel = mediaTemplateTuple.CompressionLevel, + }); } } } -- cgit v1.2.3-55-g6feb