[lldb-dev] Renaming lldb_private::Error

[Zachary Turner via lldb-dev]

I have a patch locally that renames lldb_private::Error to
lldb_private::LLDBError.  As you might expect, this is a pretty large patch
so I don't plan to put it up for review, but since it's kind of a
fundamental class I figured I should at least start a discussion.

The primary motivation for this is to enable cleaner interop between lldb's
Error class and llvm::Error.  Currently there is an ambiguity between the
two.  Obviously they are scoped in different namespaces, but it's still
confusing when looking at code to see such similarly named classes.

There are a number of advantages to llvm::Error over lldb Error which I'm
happy to elaborate on if anyone needs some clarification (I've also cc'ed
lang on here, who wrote the original llvm Error implementation).

Long term I would eventually like to deprecate lldb's Error class entirely,
and switch entirely to llvm::Error.  An intermediate transition phase would
likely involve making something like LLDBWrapError which interoperates with
llvm::Error and wraps an lldb_private::LLDBError.

For now though, I'm only proposing a very simple rename with minimal
invasiveness.

Comments?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20170429/458d4a54/attachment.html>