labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
In D80807#2063196 <https://reviews.llvm.org/D80807#2063196>, @friss wrote: > I would have committed this right away if it weren't for the slight change in > behavior I wanted to point out. With this patch, if an input string ends with > a `-`, it won't be consumed anymore. I suppose it doesn't matter. That looks fine. A trailing `-` looks like someone wanted to add more bytes but then pressed return too early, so rejecting that sounds perfectly reasonable. Since you've obviously been looking at this code lately, I've also added you the to D80755 <https://reviews.llvm.org/D80755>, for the mac perspective of accepting "uuid"s with different lengths. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80807/new/ https://reviews.llvm.org/D80807 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
