Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: installer/src/installer-lib/test/test-installer-lib-ca.cpp

Issue 29329510: Issue #1186 - Style conformity for names in installer tests (Closed)
Patch Set: Created Oct. 29, 2015, 7:26 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: installer/src/installer-lib/test/test-installer-lib-ca.cpp
===================================================================
--- a/installer/src/installer-lib/test/test-installer-lib-ca.cpp
+++ b/installer/src/installer-lib/test/test-installer-lib-ca.cpp
@@ -1,5 +1,5 @@
/**
- * \file abp_ca.cpp Top-level source for custom actions. Includes DLL initialization.
+ * \file test-installer-lib-ca.cpp Top-level source for custom actions, test version
*/
#include "DLL.h"
#include <stdexcept>
@@ -7,7 +7,7 @@
/**
* DllMain is the standard entry point call when the DLL is loaded or unloaded.
*
- * \param[in] module_handle
+ * \param[in] moduleHandle
* Handle for this instance of the DLL; same as the module handle.
* This handle allows us to get the DLL file name for logging.
* \param[in] reason
@@ -20,7 +20,7 @@
* Documentation on DLL entry points in Windows.
*/
extern "C" BOOL WINAPI DllMain(
- IN HINSTANCE module_handle,
+ IN HINSTANCE moduleHandle,
IN ULONG reason,
IN LPVOID reserved)
{
@@ -32,7 +32,7 @@
case DLL_PROCESS_ATTACH:
try
{
- DllModule::Attach(module_handle);
+ DllModule::Attach(moduleHandle);
return TRUE;
}
catch (...)

Powered by Google App Engine
This is Rietveld