From 1f81ebcbae0a319a9a62c3a29b4519e4ea40333e Mon Sep 17 00:00:00 2001
From: "Michael J. Spencer" <bigcheesegs@gmail.com>
Date: Fri, 15 Mar 2013 19:25:47 +0000
Subject: [PATCH] [Support][Path][Windows] Fix dangling else. Don't call
 CloseHandle when CloseFD is false.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@177175 91177308-0d34-0410-b5e6-96231b3b80d8
---
 lib/Support/Windows/PathV2.inc | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/lib/Support/Windows/PathV2.inc b/lib/Support/Windows/PathV2.inc
index 0f657bf3b95..23f3d14f91f 100644
--- a/lib/Support/Windows/PathV2.inc
+++ b/lib/Support/Windows/PathV2.inc
@@ -715,10 +715,10 @@ error_code mapped_file_region::init(int FD, bool CloseFD, uint64_t Offset) {
   FileDescriptor = FD;
   // Make sure that the requested size fits within SIZE_T.
   if (Size > std::numeric_limits<SIZE_T>::max()) {
-    if (FileDescriptor)
+    if (FileDescriptor) {
       if (CloseFD)
         _close(FileDescriptor);
-    else
+    } else
       ::CloseHandle(FileHandle);
     return make_error_code(errc::invalid_argument);
   }
@@ -739,10 +739,10 @@ error_code mapped_file_region::init(int FD, bool CloseFD, uint64_t Offset) {
                                           0);
   if (FileMappingHandle == NULL) {
     error_code ec = windows_error(GetLastError());
-    if (FileDescriptor)
+    if (FileDescriptor) {
       if (CloseFD)
         _close(FileDescriptor);
-    else
+    } else
       ::CloseHandle(FileHandle);
     return ec;
   }
@@ -762,10 +762,10 @@ error_code mapped_file_region::init(int FD, bool CloseFD, uint64_t Offset) {
   if (Mapping == NULL) {
     error_code ec = windows_error(GetLastError());
     ::CloseHandle(FileMappingHandle);
-    if (FileDescriptor)
+    if (FileDescriptor) {
       if (CloseFD)
         _close(FileDescriptor);
-    else
+    } else
       ::CloseHandle(FileHandle);
     return ec;
   }
@@ -777,10 +777,10 @@ error_code mapped_file_region::init(int FD, bool CloseFD, uint64_t Offset) {
       error_code ec = windows_error(GetLastError());
       ::UnmapViewOfFile(Mapping);
       ::CloseHandle(FileMappingHandle);
-      if (FileDescriptor)
+      if (FileDescriptor) {
         if (CloseFD)
           _close(FileDescriptor);
-      else
+      } else
         ::CloseHandle(FileHandle);
       return ec;
     }
@@ -790,10 +790,10 @@ error_code mapped_file_region::init(int FD, bool CloseFD, uint64_t Offset) {
   // Close all the handles except for the view. It will keep the other handles
   // alive.
   ::CloseHandle(FileMappingHandle);
-  if (FileDescriptor)
+  if (FileDescriptor) {
     if (CloseFD)
       _close(FileDescriptor); // Also closes FileHandle.
-  else
+  } else
     ::CloseHandle(FileHandle);
   return error_code::success();
 }