[Lldb-commits] [PATCH] D48633: UUID: Add support for arbitrary-sized module IDs
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jun 28 07:39:54 PDT 2018
labath added inline comments.
================
Comment at: source/Interpreter/OptionValueUUID.cpp:82
+ llvm::SmallVector<uint8_t, 20> uuid_bytes;
+ UUID::DecodeUUIDBytesFromString(s, uuid_bytes);
for (size_t i = 0; i < num_modules; ++i) {
----------------
clayborg wrote:
> Probably should have a return value that indicates success or not. What if the user enters "aab". The string must have an event number of hex digits.
The function returns the unparsed portion of the string (so empty string means success), but this code wasn't checking it for some reason.
I've added a check for that.
https://reviews.llvm.org/D48633
More information about the lldb-commits
mailing list