[all-commits] [llvm/llvm-project] f279e5: [lldb] [Communication] Add a WriteAll() method tha...

Michał Górny via All-commits all-commits at lists.llvm.org
Tue Oct 26 03:46:01 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f279e50fd0f0035e0205f4d36e7e5a8e0112fc24
      https://github.com/llvm/llvm-project/commit/f279e50fd0f0035e0205f4d36e7e5a8e0112fc24
  Author: Michał Górny <mgorny at moritz.systems>
  Date:   2021-10-26 (Tue, 26 Oct 2021)

  Changed paths:
    M lldb/include/lldb/Core/Communication.h
    M lldb/source/Core/Communication.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp
    M lldb/unittests/Core/CommunicationTest.cpp

  Log Message:
  -----------
  [lldb] [Communication] Add a WriteAll() method that resumes writing

Add a Communication::WriteAll() that resumes Write() if the initial call
did not write all data.  Use it in GDBRemoteCommunication when sending
packets in order to fix handling partial writes (i.e. just resume/retry
them rather than erring out).  This fixes LLDB failures when writing
large packets to a pty.

Differential Revision: https://reviews.llvm.org/D112169




More information about the All-commits mailing list