[Lldb-commits] [PATCH] D131615: [LLDB][NFC] Reliability fixes for IOHandlerCursesGUI
Slava Gurevich via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 11 18:30:04 PDT 2022
fixathon marked 3 inline comments as done.
fixathon added a comment.
Addressed the comments. Will deal with the signed/unsigned issue in a separate diff if still needed.
================
Comment at: lldb/source/Core/IOHandlerCursesGUI.cpp:2582
surface.MoveCursor(0, 1);
surface.HorizontalLine(surface.GetWidth());
}
----------------
clayborg wrote:
> fixathon wrote:
> > Here we have HorizontalLine() function that requires non-negative input
> But surface.HorizontalLine(...) function takes an "int", and now we are passing it a uint32_t. If this function requires a >= 0 value, it should do the check internally
I agree. I've reverted the function signatures back to signed int and will revisit the signed/unsigned issue in another diff.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131615/new/
https://reviews.llvm.org/D131615
More information about the lldb-commits
mailing list