JDevlieghere added a comment. In D67523#1668608 <https://reviews.llvm.org/D67523#1668608>, @jingham wrote:
> It seems weird to me to have the GDBRemotePacket class live in the repro > namespace. There's nothing particularly Reproducer specific about it, and it > clearly gets used outside the reproducer as well. Wouldn't it make more > sense to have this in Utils? Yep, I agree, and I considered alternatives but couldn't find a better solution. It's seems too specific for a utility class in itself and doesn't belong in Core or Target either. It really should be part of the GDB plugin, but then we can't access it without layering violations from Command... Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67523/new/ https://reviews.llvm.org/D67523 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
