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.

1 files changed, 6 insertions, 10 deletions

diff --git a/src/WixToolset.Core/Compiler_EmbeddedUI.cs b/src/WixToolset.Core/Compiler_EmbeddedUI.cs
index e71c2f56..706fe6e9 100644
--- a/src/WixToolset.Core/Compiler_EmbeddedUI.cs
+++ b/src/WixToolset.Core/Compiler_EmbeddedUI.cs
@@ -42,7 +42,7 @@ namespace WixToolset.Core
                         }
                         source = this.Core.GetAttributeIdentifierValue(sourceLineNumbers, attrib);
                         type = 0x2;
-                        this.Core.CreateSimpleReference(sourceLineNumbers, "Binary", source); // add a reference to the appropriate Binary
+                        this.Core.CreateSimpleReference(sourceLineNumbers, TupleDefinitions.Binary, source); // add a reference to the appropriate Binary
                         break;
                     case "CommandLine":
                         commandLine = this.Core.GetAttributeValue(sourceLineNumbers, attrib);
@@ -54,7 +54,7 @@ namespace WixToolset.Core
                         }
                         source = this.Core.GetAttributeIdentifierValue(sourceLineNumbers, attrib);
                         type = 0x12;
-                        this.Core.CreateSimpleReference(sourceLineNumbers, "File", source); // add a reference to the appropriate File
+                        this.Core.CreateSimpleReference(sourceLineNumbers, TupleDefinitions.File, source); // add a reference to the appropriate File
                         break;
                     case "PropertySource":
                         if (null != source)
@@ -317,16 +317,14 @@ namespace WixToolset.Core
 
             if (!this.Core.EncounteredError)
             {
-                var tuple = new MsiEmbeddedUITuple(sourceLineNumbers, id)
+                this.Core.AddTuple(new MsiEmbeddedUITuple(sourceLineNumbers, id)
                 {
                     FileName = name,
                     EntryPoint = true,
                     SupportsBasicUI = supportsBasicUI,
                     MessageFilter = messageFilter,
                     Source = sourceFile
-                };
-
-                this.Core.AddTuple(tuple);
+                });
             }
         }
 
@@ -406,13 +404,11 @@ namespace WixToolset.Core
 
             if (!this.Core.EncounteredError)
             {
-                var tuple = new MsiEmbeddedUITuple(sourceLineNumbers, id)
+                this.Core.AddTuple(new MsiEmbeddedUITuple(sourceLineNumbers, id)
                 {
                     FileName = name,
                     Source = sourceFile
-                };
-
-                this.Core.AddTuple(tuple);
+                });
             }
         }
     }