Re: [Openvpn-devel] [PATCH 06/12] openvpnmsica: Simplify static function names

2020-03-23 Thread Lev Stipakov
Stared and the code, compiled with MSVC. Change makes sense.

Acked-by: Lev Stipakov 


___
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel


[Openvpn-devel] [PATCH 06/12] openvpnmsica: Simplify static function names

2020-03-09 Thread Simon Rozman
Signed-off-by: Simon Rozman 
---
 src/openvpnmsica/openvpnmsica.c | 63 +++--
 1 file changed, 37 insertions(+), 26 deletions(-)

diff --git a/src/openvpnmsica/openvpnmsica.c b/src/openvpnmsica/openvpnmsica.c
index 4c186b13..749dce56 100644
--- a/src/openvpnmsica/openvpnmsica.c
+++ b/src/openvpnmsica/openvpnmsica.c
@@ -72,7 +72,7 @@
  * @return ERROR_SUCCESS on success; An error code otherwise
  */
 static UINT
-openvpnmsica_setup_sequence(
+setup_sequence(
 _In_ MSIHANDLE hInstall,
 _In_z_ LPCTSTR szProperty,
 _In_ struct msica_arg_seq *seq)
@@ -101,7 +101,7 @@ openvpnmsica_setup_sequence(
  *title.
  */
 static void
-_openvpnmsica_debug_popup(_In_z_ LPCTSTR szFunctionName)
+_debug_popup(_In_z_ LPCTSTR szFunctionName)
 {
 TCHAR szTitle[0x100], szMessage[0x100+MAX_PATH], szProcessPath[MAX_PATH];
 
@@ -129,9 +129,9 @@ _openvpnmsica_debug_popup(_In_z_ LPCTSTR szFunctionName)
 MessageBox(NULL, szMessage, szTitle, MB_OK);
 }
 
-#define openvpnmsica_debug_popup(f) _openvpnmsica_debug_popup(f)
+#define debug_popup(f) _debug_popup(f)
 #else  /* ifdef _DEBUG */
-#define openvpnmsica_debug_popup(f)
+#define debug_popup(f)
 #endif /* ifdef _DEBUG */
 
 
@@ -146,7 +146,7 @@ _openvpnmsica_debug_popup(_In_z_ LPCTSTR szFunctionName)
  * See: 
https://msdn.microsoft.com/en-us/library/windows/desktop/aa368072.aspx
  */
 static UINT
-openvpnmsica_set_openvpnserv_state(_In_ MSIHANDLE hInstall)
+set_openvpnserv_state(_In_ MSIHANDLE hInstall)
 {
 UINT uiResult;
 
@@ -255,13 +255,13 @@ FindSystemInfo(_In_ MSIHANDLE hInstall)
 #pragma comment(linker, DLLEXP_EXPORT)
 #endif
 
-openvpnmsica_debug_popup(TEXT(__FUNCTION__));
+debug_popup(TEXT(__FUNCTION__));
 
 BOOL bIsCoInitialized = SUCCEEDED(CoInitialize(NULL));
 
 OPENVPNMSICA_SAVE_MSI_SESSION(hInstall);
 
-openvpnmsica_set_openvpnserv_state(hInstall);
+set_openvpnserv_state(hInstall);
 
 if (bIsCoInitialized)
 {
@@ -278,7 +278,7 @@ FindTAPInterfaces(_In_ MSIHANDLE hInstall)
 #pragma comment(linker, DLLEXP_EXPORT)
 #endif
 
-openvpnmsica_debug_popup(TEXT(__FUNCTION__));
+debug_popup(TEXT(__FUNCTION__));
 
 UINT uiResult;
 BOOL bIsCoInitialized = SUCCEEDED(CoInitialize(NULL));
@@ -444,7 +444,7 @@ CloseOpenVPNGUI(_In_ MSIHANDLE hInstall)
 #endif
 UNREFERENCED_PARAMETER(hInstall); /* This CA is does not interact with MSI 
session (report errors, access properties, tables, etc.). */
 
-openvpnmsica_debug_popup(TEXT(__FUNCTION__));
+debug_popup(TEXT(__FUNCTION__));
 
 /* Find OpenVPN GUI window. */
 HWND hWnd = FindWindow(TEXT("OpenVPN-GUI"), NULL);
@@ -466,7 +466,7 @@ StartOpenVPNGUI(_In_ MSIHANDLE hInstall)
 #pragma comment(linker, DLLEXP_EXPORT)
 #endif
 
-openvpnmsica_debug_popup(TEXT(__FUNCTION__));
+debug_popup(TEXT(__FUNCTION__));
 
 UINT uiResult;
 BOOL bIsCoInitialized = SUCCEEDED(CoInitialize(NULL));
@@ -564,7 +564,11 @@ cleanup_CoInitialize:
  * @return ERROR_SUCCESS on success; An error code otherwise
  */
 static DWORD
-openvpnmsica_schedule_interface_create(_Inout_ struct msica_arg_seq *seq, 
_Inout_opt_ struct msica_arg_seq *seqRollback, _In_z_ LPCTSTR szDisplayName, 
_Inout_ int *iTicks)
+schedule_interface_create(
+_Inout_ struct msica_arg_seq *seq,
+_Inout_opt_ struct msica_arg_seq *seqRollback,
+_In_z_ LPCTSTR szDisplayName,
+_Inout_ int *iTicks)
 {
 /* Get all available network interfaces. */
 struct tap_interface_node *pInterfaceList = NULL;
@@ -658,7 +662,12 @@ cleanup_pInterfaceList:
  * @return ERROR_SUCCESS on success; An error code otherwise
  */
 static DWORD
-openvpnmsica_schedule_interface_delete(_Inout_ struct msica_arg_seq *seq, 
_Inout_opt_ struct msica_arg_seq *seqCommit, _Inout_opt_ struct msica_arg_seq 
*seqRollback, _In_z_ LPCTSTR szDisplayName, _Inout_ int *iTicks)
+schedule_interface_delete(
+_Inout_ struct msica_arg_seq *seq,
+_Inout_opt_ struct msica_arg_seq *seqCommit,
+_Inout_opt_ struct msica_arg_seq *seqRollback,
+_In_z_ LPCTSTR szDisplayName,
+_Inout_ int *iTicks)
 {
 /* Get available TUN/TAP interfaces. */
 struct tap_interface_node *pInterfaceList = NULL;
@@ -725,7 +734,7 @@ EvaluateTAPInterfaces(_In_ MSIHANDLE hInstall)
 #pragma comment(linker, DLLEXP_EXPORT)
 #endif
 
-openvpnmsica_debug_popup(TEXT(__FUNCTION__));
+debug_popup(TEXT(__FUNCTION__));
 
 UINT uiResult;
 BOOL bIsCoInitialized = SUCCEEDED(CoInitialize(NULL));
@@ -887,7 +896,7 @@ EvaluateTAPInterfaces(_In_ MSIHANDLE hInstall)
 free(szValue);
 
 /* Component is or should be installed. Schedule interface 
creation. */
-if (openvpnmsica_schedule_interface_create(
+if (schedule_interface_create(
 ,
 bRollbackEnabled ?  : 
NULL,
 szDisplayNameEx,
@@ -904,7 +913,7 @@ EvaluateTAPInterfaces(_In_ MSIHANDLE hInstall)