Simplify BootstrapperApplication header files

Take this breaking change opportunity to simplify the header files used by
BA devs.

18 files changed, 42 insertions, 91 deletions

diff --git a/src/api/burn/WixToolset.BootstrapperApplicationApi/WixToolset.BootstrapperApplicationApi.nuspec b/src/api/burn/WixToolset.BootstrapperApplicationApi/WixToolset.BootstrapperApplicationApi.nuspec
index 22099635..4c850cdf 100644
--- a/src/api/burn/WixToolset.BootstrapperApplicationApi/WixToolset.BootstrapperApplicationApi.nuspec
+++ b/src/api/burn/WixToolset.BootstrapperApplicationApi/WixToolset.BootstrapperApplicationApi.nuspec
@@ -37,8 +37,8 @@
     <file src="v143\x86\mbanative.dll" target="runtimes\win-x86\native" />
 
     <file src="$projectFolder$\..\balutil\inc\*" target="build\native\include" />
-    <file src="$projectFolder$\..\inc\BootstrapperApplication.h" target="build\native\include" />
-    <file src="$projectFolder$\..\inc\BootstrapperEngine.h" target="build\native\include" />
+    <file src="$projectFolder$\..\inc\BootstrapperApplicationTypes.h" target="build\native\include" />
+    <file src="$projectFolder$\..\inc\BootstrapperEngineTypes.h" target="build\native\include" />
     <file src="v141\x86\balutil.lib" target="build\native\v14\x86" />
     <file src="v141\x64\balutil.lib" target="build\native\v14\x64" />
     <file src="v141\ARM64\balutil.lib" target="build\native\v14\ARM64" />
diff --git a/src/api/burn/balutil/balutil.cpp b/src/api/burn/balutil/balutil.cpp
index 7f774b6c..5d8bfa9c 100644
--- a/src/api/burn/balutil/balutil.cpp
+++ b/src/api/burn/balutil/balutil.cpp
@@ -38,7 +38,7 @@ static void MsgProc(
 
 // prototypes
 
-DAPI_(HRESULT) BootstrapperApplicationRun(
+EXTERN_C HRESULT __stdcall BootstrapperApplicationRun(
     __in IBootstrapperApplication* pApplication
     )
 {
diff --git a/src/api/burn/balutil/balutil.vcxproj b/src/api/burn/balutil/balutil.vcxproj
index 7d539b5b..2b0fd23f 100644
--- a/src/api/burn/balutil/balutil.vcxproj
+++ b/src/api/burn/balutil/balutil.vcxproj
@@ -63,7 +63,7 @@
     <ClCompile Include="msg.cpp" />
   </ItemGroup>
   <ItemGroup>
-    <ClInclude Include="inc\BalBaseBootstrapperApplication.h" />
+    <ClInclude Include="inc\BootstrapperApplicationBase.h" />
     <ClInclude Include="inc\balcondition.h" />
     <ClInclude Include="inc\balinfo.h" />
     <ClInclude Include="inc\balretry.h" />
diff --git a/src/api/burn/balutil/inc/BootstrapperApplication.h b/src/api/burn/balutil/inc/BootstrapperApplication.h
new file mode 100644
index 00000000..d11fccf0
--- /dev/null
+++ b/src/api/burn/balutil/inc/BootstrapperApplication.h
@@ -0,0 +1,21 @@
+#pragma once
+// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
+
+#include "IBootstrapperApplication.h"
+
+#if defined(__cplusplus)
+extern "C" {
+#endif
+
+/*******************************************************************
+ BootstrapperApplicationRun - runs the IBootstrapperApplication until
+                              the application quits.
+
+********************************************************************/
+HRESULT __stdcall BootstrapperApplicationRun(
+    __in IBootstrapperApplication* pApplication
+    );
+
+#if defined(__cplusplus)
+}
+#endif
diff --git a/src/api/burn/balutil/inc/BalBaseBootstrapperApplication.h b/src/api/burn/balutil/inc/BootstrapperApplicationBase.h
index 09f1f7a2..a3a3643f 100644
--- a/src/api/burn/balutil/inc/BalBaseBootstrapperApplication.h
+++ b/src/api/burn/balutil/inc/BootstrapperApplicationBase.h
@@ -9,9 +9,9 @@
 #include "balinfo.h"
 #include "balretry.h"
 
-#define CBalBaseBootstrapperApplication CBootstrapperApplication
+#define CBalBaseBootstrapperApplication CBootstrapperApplicationBase
 
-class CBootstrapperApplication : public IBootstrapperApplication
+class CBootstrapperApplicationBase : public IBootstrapperApplication
 {
 public: // IUnknown
     virtual STDMETHODIMP QueryInterface(
@@ -1187,7 +1187,7 @@ protected:
         return m_fCanceled;
     }
 
-    CBalBaseBootstrapperApplication(
+    CBootstrapperApplicationBase(
         __in DWORD dwRetryCount = 0,
         __in DWORD dwRetryTimeout = 1000
         )
@@ -1208,7 +1208,7 @@ protected:
         BalRetryInitialize(dwRetryCount, dwRetryTimeout);
     }
 
-    virtual ~CBalBaseBootstrapperApplication()
+    virtual ~CBootstrapperApplicationBase()
     {
         BalInfoUninitializeCommandLine(&m_BalInfoCommand);
         BalRetryUninitialize();
diff --git a/src/api/burn/balutil/inc/IBootstrapperApplication.h b/src/api/burn/balutil/inc/IBootstrapperApplication.h
index 3f57f095..a08f6863 100644
--- a/src/api/burn/balutil/inc/IBootstrapperApplication.h
+++ b/src/api/burn/balutil/inc/IBootstrapperApplication.h
@@ -1,8 +1,8 @@
 #pragma once
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
-#include <BootstrapperApplication.h>
-#include <IBootstrapperEngine.h>
+#include "BootstrapperApplicationTypes.h"
+#include "IBootstrapperEngine.h"
 
 DECLARE_INTERFACE_IID_(IBootstrapperApplication, IUnknown, "53C31D56-49C0-426B-AB06-099D717C67FE")
 {
diff --git a/src/api/burn/balutil/inc/IBootstrapperEngine.h b/src/api/burn/balutil/inc/IBootstrapperEngine.h
index ef3b9812..02094aa5 100644
--- a/src/api/burn/balutil/inc/IBootstrapperEngine.h
+++ b/src/api/burn/balutil/inc/IBootstrapperEngine.h
@@ -1,7 +1,7 @@
 #pragma once
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
-#include "BootstrapperEngine.h"
+#include "BootstrapperEngineTypes.h"
 
 DECLARE_INTERFACE_IID_(IBootstrapperEngine, IUnknown, "6480D616-27A0-44D7-905B-81512C29C2FB")
 {
diff --git a/src/api/burn/balutil/inc/balutil.h b/src/api/burn/balutil/inc/balutil.h
index d77b3599..32190b56 100644
--- a/src/api/burn/balutil/inc/balutil.h
+++ b/src/api/burn/balutil/inc/balutil.h
@@ -43,15 +43,6 @@ static const HRESULT E_PREREQBA_INFINITE_LOOP = MAKE_HRESULT(SEVERITY_ERROR, FAC
 
 
 /*******************************************************************
- BootstrapperApplicationRun - runs the IBootstrapperApplication until
-                              the application quits.
-
-********************************************************************/
-DAPI_(HRESULT) BootstrapperApplicationRun(
-    __in IBootstrapperApplication* pApplication
-    );
-
-/*******************************************************************
  BalDebuggerCheck - allows bootstrapper application to explicitly check
         whether a debugger should be attached to the boostrapper application.
 
diff --git a/src/api/burn/balutil/precomp.h b/src/api/burn/balutil/precomp.h
index 753bc21f..3123143e 100644
--- a/src/api/burn/balutil/precomp.h
+++ b/src/api/burn/balutil/precomp.h
@@ -30,7 +30,7 @@
 #include <thmutil.h>
 #include <xmlutil.h>
 
-#include "IBootstrapperApplication.h"
+#include "BootstrapperApplication.h"
 
 #include "balutil.h"
 #include "BalBootstrapperEngine.h"
diff --git a/src/api/burn/bextutil/bextutil.nuspec b/src/api/burn/bextutil/bextutil.nuspec
index c710f424..436c0028 100644
--- a/src/api/burn/bextutil/bextutil.nuspec
+++ b/src/api/burn/bextutil/bextutil.nuspec
@@ -21,8 +21,8 @@
     <file src="$projectFolder$\build\$id$.props" target="build\" />
     <file src="$projectFolder$\..\..\..\internal\images\wix.png" />
     <file src="$projectFolder$\inc\*" target="build\native\include" />
-    <file src="$projectFolder$\..\inc\BootstrapperExtension.h" target="build\native\include" />
-    <file src="$projectFolder$\..\inc\BootstrapperExtensionEngine.h" target="build\native\include" />
+    <file src="$projectFolder$\..\inc\BootstrapperExtensionTypes.h" target="build\native\include" />
+    <file src="$projectFolder$\..\inc\BootstrapperExtensionEngineTypes.h" target="build\native\include" />
     <file src="..\..\v141\x86\bextutil.lib" target="build\native\v14\x86" />
     <file src="..\..\v141\x64\bextutil.lib" target="build\native\v14\x64" />
     <file src="..\..\v141\ARM64\bextutil.lib" target="build\native\v14\ARM64" />
diff --git a/src/api/burn/bextutil/inc/IBootstrapperExtension.h b/src/api/burn/bextutil/inc/IBootstrapperExtension.h
index 4005a9fd..19df0516 100644
--- a/src/api/burn/bextutil/inc/IBootstrapperExtension.h
+++ b/src/api/burn/bextutil/inc/IBootstrapperExtension.h
@@ -1,7 +1,7 @@
 #pragma once
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
-#include <BootstrapperExtension.h>
+#include "BootstrapperExtensionTypes.h"
 
 DECLARE_INTERFACE_IID_(IBootstrapperExtension, IUnknown, "93123C9D-796B-4FCD-A507-6EDEF9A925FD")
 {
diff --git a/src/api/burn/bextutil/inc/IBootstrapperExtensionEngine.h b/src/api/burn/bextutil/inc/IBootstrapperExtensionEngine.h
index b23a57b7..a78868f2 100644
--- a/src/api/burn/bextutil/inc/IBootstrapperExtensionEngine.h
+++ b/src/api/burn/bextutil/inc/IBootstrapperExtensionEngine.h
@@ -1,7 +1,7 @@
 #pragma once
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
-#include <BootstrapperExtensionEngine.h>
+#include "BootstrapperExtensionEngineTypes.h"
 
 DECLARE_INTERFACE_IID_(IBootstrapperExtensionEngine, IUnknown, "9D027A39-F6B6-42CC-9737-C185089EB263")
 {
diff --git a/src/api/burn/inc/BootstrapperApplication.h b/src/api/burn/inc/BootstrapperApplicationTypes.h
index 8b945b77..c7eaa3d3 100644
--- a/src/api/burn/inc/BootstrapperApplication.h
+++ b/src/api/burn/inc/BootstrapperApplicationTypes.h
@@ -1,7 +1,7 @@
 #pragma once
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
-#include "BootstrapperEngine.h"
+#include "BootstrapperEngineTypes.h"
 
 #if defined(__cplusplus)
 extern "C" {
@@ -1535,57 +1535,6 @@ struct BA_ONUNREGISTERCOMPLETE_RESULTS
     DWORD dwApiVersion;
 };
 
-#ifdef TODO_DELETE
-
-extern "C" typedef HRESULT(WINAPI *PFN_BOOTSTRAPPER_APPLICATION_PROC)(
-    __in BOOTSTRAPPER_APPLICATION_MESSAGE message,
-    __in const LPVOID pvArgs,
-    __inout LPVOID pvResults,
-    __in_opt LPVOID pvContext
-    );
-
-struct BOOTSTRAPPER_DESTROY_ARGS
-{
-    DWORD dwApiVersion;
-    BOOL fReload;
-};
-
-struct BOOTSTRAPPER_DESTROY_RESULTS
-{
-    DWORD dwApiVersion;
-    BOOL fDisableUnloading; // indicates the BA dll must not be unloaded after BootstrapperApplicationDestroy.
-};
-
-extern "C" typedef void (WINAPI *PFN_BOOTSTRAPPER_APPLICATION_DESTROY)(
-    __in const BOOTSTRAPPER_DESTROY_ARGS* pArgs,
-    __inout BOOTSTRAPPER_DESTROY_RESULTS* pResults
-    );
-
-
-
-struct BOOTSTRAPPER_CREATE_ARGS
-{
-    DWORD dwApiVersion;
-    DWORD64 qwEngineAPIVersion;
-    PFN_BOOTSTRAPPER_ENGINE_PROC pfnBootstrapperEngineProc;
-    LPVOID pvBootstrapperEngineProcContext;
-    BOOTSTRAPPER_COMMAND* pCommand;
-};
-
-struct BOOTSTRAPPER_CREATE_RESULTS
-{
-    DWORD dwApiVersion;
-    PFN_BOOTSTRAPPER_APPLICATION_PROC pfnBootstrapperApplicationProc;
-    LPVOID pvBootstrapperApplicationProcContext;
-};
-
-extern "C" typedef HRESULT(WINAPI *PFN_BOOTSTRAPPER_APPLICATION_CREATE)(
-    __in const BOOTSTRAPPER_CREATE_ARGS* pArgs,
-    __inout BOOTSTRAPPER_CREATE_RESULTS* pResults
-    );
-
-#endif
-
 #if defined(__cplusplus)
 }
 #endif
diff --git a/src/api/burn/inc/BootstrapperEngine.h b/src/api/burn/inc/BootstrapperEngineTypes.h
index 7af41d97..e2634383 100644
--- a/src/api/burn/inc/BootstrapperEngine.h
+++ b/src/api/burn/inc/BootstrapperEngineTypes.h
@@ -448,13 +448,6 @@ typedef struct _BAENGINE_GETRELATEDBUNDLEVARIABLE_RESULTS
     DWORD cchValue;
 } BAENGINE_GETRELATEDBUNDLEVARIABLE_RESULTS;
 
-// extern "C" typedef HRESULT(WINAPI *PFN_BOOTSTRAPPER_ENGINE_PROC)(
-//     __in BOOTSTRAPPER_ENGINE_MESSAGE message,
-//     __in const LPVOID pvArgs,
-//     __inout LPVOID pvResults,
-//     __in_opt LPVOID pvContext
-//     );
-
 #if defined(__cplusplus)
 }
 #endif
diff --git a/src/api/burn/inc/BootstrapperExtensionEngine.h b/src/api/burn/inc/BootstrapperExtensionEngineTypes.h
index 24c304c6..24c304c6 100644
--- a/src/api/burn/inc/BootstrapperExtensionEngine.h
+++ b/src/api/burn/inc/BootstrapperExtensionEngineTypes.h
diff --git a/src/api/burn/inc/BootstrapperExtension.h b/src/api/burn/inc/BootstrapperExtensionTypes.h
index b1fa6408..4d45b59d 100644
--- a/src/api/burn/inc/BootstrapperExtension.h
+++ b/src/api/burn/inc/BootstrapperExtensionTypes.h
@@ -1,7 +1,7 @@
 #pragma once
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
-#include <BootstrapperExtensionEngine.h>
+#include "BootstrapperExtensionEngineTypes.h"
 
 #if defined(__cplusplus)
 extern "C" {
diff --git a/src/api/burn/test/BalUtilUnitTest/TestBootstrapperApplication.cpp b/src/api/burn/test/BalUtilUnitTest/TestBootstrapperApplication.cpp
index b345ab9f..d0b38064 100644
--- a/src/api/burn/test/BalUtilUnitTest/TestBootstrapperApplication.cpp
+++ b/src/api/burn/test/BalUtilUnitTest/TestBootstrapperApplication.cpp
@@ -1,12 +1,11 @@
 // Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
 #include "precomp.h"
-#include "BalBaseBootstrapperApplication.h"
 
-class CTestBootstrapperApplication : public CBalBaseBootstrapperApplication
+class CTestBootstrapperApplication : public CBootstrapperApplicationBase
 {
 public:
-    CTestBootstrapperApplication() : CBalBaseBootstrapperApplication()
+    CTestBootstrapperApplication() : CBootstrapperApplicationBase()
     {
     }
 };
diff --git a/src/api/burn/test/BalUtilUnitTest/precomp.h b/src/api/burn/test/BalUtilUnitTest/precomp.h
index 78add593..0edf4664 100644
--- a/src/api/burn/test/BalUtilUnitTest/precomp.h
+++ b/src/api/burn/test/BalUtilUnitTest/precomp.h
@@ -15,9 +15,7 @@
 #include <dutil.h>
 #include <dictutil.h>
 
-#include <IBootstrapperApplication.h>
-#include <balutil.h>
-#include <balretry.h>
+#include <BootstrapperApplicationBase.h>
 
 #include "TestBootstrapperApplication.h"