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 Nov. 14, 2015, 5:55 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 (...)
« no previous file with comments | « installer/src/installer-lib/test/property_test.cpp ('k') | installer/src/installer-lib/test/test-installer-lib-sandbox.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld