From 00cae20654be83a7b4db615e77c29c943374da31 Mon Sep 17 00:00:00 2001
From: Doyle Thai <dyl.thai@gmail.com>
Date: Mon, 8 May 2017 17:59:51 +1000
Subject: [PATCH] Rename file flags/actions to be consistent

---
 dqn.h             | 28 ++++++++++++++--------------
 dqn_unit_test.cpp |  8 ++++----
 2 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/dqn.h b/dqn.h
index 29de7c2..d8fbf57 100644
--- a/dqn.h
+++ b/dqn.h
@@ -640,25 +640,25 @@ DQN_FILE_SCOPE i32     Dqn_I32ToWStr(i32 value, wchar_t *buf, i32 bufSize);
 ////////////////////////////////////////////////////////////////////////////////
 enum DqnFilePermissionFlag
 {
-	dqnfilepermissionflag_read    = (1 << 0),
-	dqnfilepermissionflag_write   = (1 << 1),
-	dqnfilepermissionflag_execute = (1 << 2),
-	dqnfilepermissionflag_all     = (1 << 3)
+	DqnFilePermissionFlag_Read    = (1 << 0),
+	DqnFilePermissionFlag_Write   = (1 << 1),
+	DqnFilePermissionFlag_Execute = (1 << 2),
+	DqnFilePermissionFlag_All     = (1 << 3)
 };
 
 enum DqnFileAction
 {
 	// Only open file if it exists. Fails and returns false if file did not
 	// exist or could not open.
-	dqnfileaction_open_only,
+	DqnFileAction_OpenOnly,
 
 	// Try and create file. Return true if it was able to create. If it already
 	// exists, this will fail.
-	dqnfileaction_create_if_not_exist,
+	DqnFileAction_CreateIfNotExist,
 
 	// Clear the file contents to zero if it exists. Fails and returns false if
 	// file does not exist.
-	dqnfileaction_clear_if_exist,
+	DqnFileAction_ClearIfExist,
 };
 
 typedef struct DqnFile
@@ -3031,15 +3031,15 @@ FILE_SCOPE bool DqnFile_OpenInternal(const wchar_t *const path,
 
 #ifdef DQN_WIN32_IMPLEMENTATION
 	DWORD win32Permission = 0;
-	if (permissionFlags & dqnfilepermissionflag_all)
+	if (permissionFlags & DqnFilePermissionFlag_All)
 	{
 		win32Permission = GENERIC_ALL;
 	}
 	else
 	{
-		if (permissionFlags & dqnfilepermissionflag_read)    win32Permission |= GENERIC_READ;
-		if (permissionFlags & dqnfilepermissionflag_write)   win32Permission |= GENERIC_WRITE;
-		if (permissionFlags & dqnfilepermissionflag_execute) win32Permission |= GENERIC_EXECUTE;
+		if (permissionFlags & DqnFilePermissionFlag_Read)    win32Permission |= GENERIC_READ;
+		if (permissionFlags & DqnFilePermissionFlag_Write)   win32Permission |= GENERIC_WRITE;
+		if (permissionFlags & DqnFilePermissionFlag_Execute) win32Permission |= GENERIC_EXECUTE;
 	}
 
 	DWORD win32Action = 0;
@@ -3047,9 +3047,9 @@ FILE_SCOPE bool DqnFile_OpenInternal(const wchar_t *const path,
 	{
 		// Allow fall through
 		default: DQN_ASSERT(DQN_INVALID_CODE_PATH);
-		case dqnfileaction_open_only:           win32Action = OPEN_EXISTING; break;
-		case dqnfileaction_clear_if_exist:      win32Action = TRUNCATE_EXISTING; break;
-		case dqnfileaction_create_if_not_exist: win32Action = CREATE_NEW; break;
+		case DqnFileAction_OpenOnly:         win32Action = OPEN_EXISTING; break;
+		case DqnFileAction_ClearIfExist:     win32Action = TRUNCATE_EXISTING; break;
+		case DqnFileAction_CreateIfNotExist: win32Action = CREATE_NEW; break;
 	}
 
 	HANDLE handle = CreateFileW(path, win32Permission, 0, NULL, win32Action,
diff --git a/dqn_unit_test.cpp b/dqn_unit_test.cpp
index e6f9509..d5f3a0d 100644
--- a/dqn_unit_test.cpp
+++ b/dqn_unit_test.cpp
@@ -980,8 +980,8 @@ void FileTest()
 			DqnFile file = {};
 			DQN_ASSERT(DqnFile_Open(
 				".clang-format", &file,
-				(dqnfilepermissionflag_write | dqnfilepermissionflag_read),
-				dqnfileaction_open_only));
+				(DqnFilePermissionFlag_Write | DqnFilePermissionFlag_Read),
+				DqnFileAction_OpenOnly));
 			DQN_ASSERT(file.size == 1320);
 
 			u8 *buffer = (u8 *)calloc(1, (size_t)file.size * sizeof(u8));
@@ -997,8 +997,8 @@ void FileTest()
 			DqnFile file = {};
 			DQN_ASSERT(!DqnFile_Open(
 				"asdljasdnel;kajdf", &file,
-				(dqnfilepermissionflag_write | dqnfilepermissionflag_read),
-				dqnfileaction_open_only));
+				(DqnFilePermissionFlag_Write | DqnFilePermissionFlag_Read),
+				DqnFileAction_OpenOnly));
 			DQN_ASSERT(file.size == 0);
 			DQN_ASSERT(file.permissionFlags == 0);
 			DQN_ASSERT(!file.handle);