lemo added a comment.
> I'm not sure if there is a suitable place for that function. This is
> needed in "ObjectFileMachO" and two dynamic loader plugins.
Then your factory idea may be the next best thing. While we're at it, maybe
we can remove the UUID::SetBytes() from the public interface, and make the
UUID an immutable value type:
Ex. instead of:
UUID uuid;
...
uuid.SetBytes(...)
We'd have:
UUID uuid;
uuid = UUID(...);
// or
uuid = { ... };
// or
uuid = UUID::factory(...);
What do you think?
https://reviews.llvm.org/D48479
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
