[Lldb-commits] [lldb] r220473 - Fix code where goto jumped over local variable initialization.

Hafiz Abid Qadeer hafiz_abid at mentor.com
Thu Oct 23 03:36:54 PDT 2014


Author: abidh
Date: Thu Oct 23 05:36:53 2014
New Revision: 220473

URL: http://llvm.org/viewvc/llvm-project?rev=220473&view=rev
Log:
Fix code where goto jumped over local variable initialization.

Modified:
    lldb/trunk/source/Host/windows/ConnectionGenericFileWindows.cpp

Modified: lldb/trunk/source/Host/windows/ConnectionGenericFileWindows.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Host/windows/ConnectionGenericFileWindows.cpp?rev=220473&r1=220472&r2=220473&view=diff
==============================================================================
--- lldb/trunk/source/Host/windows/ConnectionGenericFileWindows.cpp (original)
+++ lldb/trunk/source/Host/windows/ConnectionGenericFileWindows.cpp Thu Oct 23 05:36:53 2014
@@ -182,6 +182,8 @@ size_t
 ConnectionGenericFile::Read(void *dst, size_t dst_len, uint32_t timeout_usec, lldb::ConnectionStatus &status, Error *error_ptr)
 {
     ReturnInfo return_info;
+    BOOL result = 0;
+    DWORD bytes_read = 0;
 
     if (error_ptr)
         error_ptr->Clear();
@@ -194,7 +196,7 @@ ConnectionGenericFile::Read(void *dst, s
 
     m_overlapped.hEvent = m_event_handles[kBytesAvailableEvent];
 
-    BOOL result = ::ReadFile(m_file, dst, dst_len, NULL, &m_overlapped);
+    result = ::ReadFile(m_file, dst, dst_len, NULL, &m_overlapped);
     if (result || ::GetLastError() == ERROR_IO_PENDING)
     {
         if (!result)
@@ -222,7 +224,6 @@ ConnectionGenericFile::Read(void *dst, s
             }
         }
         // The data is ready.  Figure out how much was read and return;
-        DWORD bytes_read = 0;
         if (!::GetOverlappedResult(m_file, &m_overlapped, &bytes_read, FALSE))
         {
             DWORD result_error = ::GetLastError();
@@ -279,6 +280,8 @@ size_t
 ConnectionGenericFile::Write(const void *src, size_t src_len, lldb::ConnectionStatus &status, Error *error_ptr)
 {
     ReturnInfo return_info;
+    DWORD bytes_written = 0;
+    BOOL result = 0;
 
     if (error_ptr)
         error_ptr->Clear();
@@ -292,14 +295,13 @@ ConnectionGenericFile::Write(const void
     m_overlapped.hEvent = NULL;
 
     // Writes are not interruptible like reads are, so just block until it's done.
-    BOOL result = ::WriteFile(m_file, src, src_len, NULL, &m_overlapped);
+    result = ::WriteFile(m_file, src, src_len, NULL, &m_overlapped);
     if (!result && ::GetLastError() != ERROR_IO_PENDING)
     {
         return_info.Set(0, eConnectionStatusError, ::GetLastError());
         goto finish;
     }
 
-    DWORD bytes_written = 0;
     if (!::GetOverlappedResult(m_file, &m_overlapped, &bytes_written, TRUE))
     {
         return_info.Set(bytes_written, eConnectionStatusError, ::GetLastError());
@@ -341,4 +343,4 @@ ConnectionGenericFile::IncrementFilePoin
     old_pos.QuadPart += amount;
     m_overlapped.Offset = old_pos.LowPart;
     m_overlapped.OffsetHigh = old_pos.HighPart;
-}
\ No newline at end of file
+}





More information about the lldb-commits mailing list