Remove unused member and simplify. NFC.
authorRafael Espindola <rafael.espindola@gmail.com>
Tue, 16 Dec 2014 02:19:26 +0000 (02:19 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Tue, 16 Dec 2014 02:19:26 +0000 (02:19 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@224309 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Support/FileSystem.h
lib/Support/Windows/Path.inc

index 4acff6623d7e6024f48bf24251e18fd039c7f2d9..d86780c38a41a5e01429ff2ddad900605f7cd7c3 100644 (file)
@@ -640,7 +640,6 @@ private:
   uint64_t Size;
   void *Mapping;
 #ifdef LLVM_ON_WIN32
-  int FileDescriptor;
   void *FileHandle;
   void *FileMappingHandle;
 #endif
index c74033dcb6b56018b9bcddf7cbe3d14d8001e35f..a9df6c0c701e264aab439a3943d8a422cce710f2 100644 (file)
@@ -460,14 +460,9 @@ std::error_code setLastModificationAndAccessTime(int FD, TimeValue Time) {
 
 std::error_code mapped_file_region::init(int FD, uint64_t Offset,
                                          mapmode Mode) {
-  FileDescriptor = FD;
   // Make sure that the requested size fits within SIZE_T.
-  if (Size > std::numeric_limits<SIZE_T>::max()) {
-    if (FileDescriptor) {
-    } else
-      ::CloseHandle(FileHandle);
+  if (Size > std::numeric_limits<SIZE_T>::max())
     return make_error_code(errc::invalid_argument);
-  }
 
   DWORD flprotect;
   switch (Mode) {
@@ -483,9 +478,6 @@ std::error_code mapped_file_region::init(int FD, uint64_t Offset,
                            0);
   if (FileMappingHandle == NULL) {
     std::error_code ec = windows_error(GetLastError());
-    if (FileDescriptor) {
-    } else
-      ::CloseHandle(FileHandle);
     return ec;
   }
 
@@ -503,9 +495,6 @@ std::error_code mapped_file_region::init(int FD, uint64_t Offset,
   if (Mapping == NULL) {
     std::error_code ec = windows_error(GetLastError());
     ::CloseHandle(FileMappingHandle);
-    if (FileDescriptor) {
-    } else
-      ::CloseHandle(FileHandle);
     return ec;
   }
 
@@ -516,9 +505,6 @@ std::error_code mapped_file_region::init(int FD, uint64_t Offset,
       std::error_code ec = windows_error(GetLastError());
       ::UnmapViewOfFile(Mapping);
       ::CloseHandle(FileMappingHandle);
-      if (FileDescriptor) {
-      } else
-        ::CloseHandle(FileHandle);
       return ec;
     }
     Size = mbi.RegionSize;
@@ -527,28 +513,23 @@ std::error_code mapped_file_region::init(int FD, uint64_t Offset,
   // Close all the handles except for the view. It will keep the other handles
   // alive.
   ::CloseHandle(FileMappingHandle);
-  if (FileDescriptor) {
-  } else
-    ::CloseHandle(FileHandle);
   return std::error_code();
 }
 
 mapped_file_region::mapped_file_region(int fd, mapmode mode, uint64_t length,
                                        uint64_t offset, std::error_code &ec)
-    : Size(length), Mapping(), FileDescriptor(fd),
+    : Size(length), Mapping(),
       FileHandle(INVALID_HANDLE_VALUE), FileMappingHandle() {
   FileHandle = reinterpret_cast<HANDLE>(_get_osfhandle(fd));
   if (FileHandle == INVALID_HANDLE_VALUE) {
-    FileDescriptor = 0;
     ec = make_error_code(errc::bad_file_descriptor);
     return;
   }
 
-  ec = init(FileDescriptor, offset, mode);
+  ec = init(fd, offset, mode);
   if (ec) {
     Mapping = FileMappingHandle = 0;
     FileHandle = INVALID_HANDLE_VALUE;
-    FileDescriptor = 0;
   }
 }