Browse Source

FileAccessWindows: Cosmetic improvements

Max Hilbrunner 4 years ago
parent
commit
a5c179e4f8
1 changed files with 9 additions and 4 deletions
  1. 9 4
      drivers/windows/file_access_windows.cpp

+ 9 - 4
drivers/windows/file_access_windows.cpp

@@ -77,8 +77,8 @@ Error FileAccessWindows::_open(const String &p_path, int p_mode_flags) {
 		return ERR_INVALID_PARAMETER;
 		return ERR_INVALID_PARAMETER;
 	}
 	}
 
 
-	/* pretty much every implementation that uses fopen as primary
-	   backend supports utf8 encoding */
+	/* Pretty much every implementation that uses fopen as primary
+	   backend supports utf8 encoding. */
 
 
 	struct _stat st;
 	struct _stat st;
 	if (_wstat((LPCWSTR)(path.utf16().get_data()), &st) == 0) {
 	if (_wstat((LPCWSTR)(path.utf16().get_data()), &st) == 0) {
@@ -157,10 +157,10 @@ void FileAccessWindows::close() {
 #else
 #else
 			if (!PathFileExistsW((LPCWSTR)(save_path.utf16().get_data()))) {
 			if (!PathFileExistsW((LPCWSTR)(save_path.utf16().get_data()))) {
 #endif
 #endif
-				//creating new file
+				// Creating new file
 				rename_error = _wrename((LPCWSTR)((save_path + ".tmp").utf16().get_data()), (LPCWSTR)(save_path.utf16().get_data())) != 0;
 				rename_error = _wrename((LPCWSTR)((save_path + ".tmp").utf16().get_data()), (LPCWSTR)(save_path.utf16().get_data())) != 0;
 			} else {
 			} else {
-				//atomic replace for existing file
+				// Atomic replace for existing file
 				rename_error = !ReplaceFileW((LPCWSTR)(save_path.utf16().get_data()), (LPCWSTR)((save_path + ".tmp").utf16().get_data()), nullptr, 2 | 4, nullptr, nullptr);
 				rename_error = !ReplaceFileW((LPCWSTR)(save_path.utf16().get_data()), (LPCWSTR)((save_path + ".tmp").utf16().get_data()), nullptr, 2 | 4, nullptr, nullptr);
 			}
 			}
 			if (rename_error) {
 			if (rename_error) {
@@ -205,6 +205,7 @@ void FileAccessWindows::seek(uint64_t p_position) {
 
 
 void FileAccessWindows::seek_end(int64_t p_position) {
 void FileAccessWindows::seek_end(int64_t p_position) {
 	ERR_FAIL_COND(!f);
 	ERR_FAIL_COND(!f);
+
 	if (_fseeki64(f, p_position, SEEK_END)) {
 	if (_fseeki64(f, p_position, SEEK_END)) {
 		check_errors();
 		check_errors();
 	}
 	}
@@ -237,6 +238,7 @@ bool FileAccessWindows::eof_reached() const {
 
 
 uint8_t FileAccessWindows::get_8() const {
 uint8_t FileAccessWindows::get_8() const {
 	ERR_FAIL_COND_V(!f, 0);
 	ERR_FAIL_COND_V(!f, 0);
+
 	if (flags == READ_WRITE || flags == WRITE_READ) {
 	if (flags == READ_WRITE || flags == WRITE_READ) {
 		if (prev_op == WRITE) {
 		if (prev_op == WRITE) {
 			fflush(f);
 			fflush(f);
@@ -273,6 +275,7 @@ Error FileAccessWindows::get_error() const {
 
 
 void FileAccessWindows::flush() {
 void FileAccessWindows::flush() {
 	ERR_FAIL_COND(!f);
 	ERR_FAIL_COND(!f);
+
 	fflush(f);
 	fflush(f);
 	if (prev_op == WRITE) {
 	if (prev_op == WRITE) {
 		prev_op = 0;
 		prev_op = 0;
@@ -281,6 +284,7 @@ void FileAccessWindows::flush() {
 
 
 void FileAccessWindows::store_8(uint8_t p_dest) {
 void FileAccessWindows::store_8(uint8_t p_dest) {
 	ERR_FAIL_COND(!f);
 	ERR_FAIL_COND(!f);
+
 	if (flags == READ_WRITE || flags == WRITE_READ) {
 	if (flags == READ_WRITE || flags == WRITE_READ) {
 		if (prev_op == READ) {
 		if (prev_op == READ) {
 			if (last_error != ERR_FILE_EOF) {
 			if (last_error != ERR_FILE_EOF) {
@@ -295,6 +299,7 @@ void FileAccessWindows::store_8(uint8_t p_dest) {
 void FileAccessWindows::store_buffer(const uint8_t *p_src, uint64_t p_length) {
 void FileAccessWindows::store_buffer(const uint8_t *p_src, uint64_t p_length) {
 	ERR_FAIL_COND(!f);
 	ERR_FAIL_COND(!f);
 	ERR_FAIL_COND(!p_src && p_length > 0);
 	ERR_FAIL_COND(!p_src && p_length > 0);
+
 	if (flags == READ_WRITE || flags == WRITE_READ) {
 	if (flags == READ_WRITE || flags == WRITE_READ) {
 		if (prev_op == READ) {
 		if (prev_op == READ) {
 			if (last_error != ERR_FILE_EOF) {
 			if (last_error != ERR_FILE_EOF) {