[Lldb-commits] [PATCH] D59235: Remove Support for DWARF64

Eric Christopher via lldb-commits lldb-commits at lists.llvm.org
Tue Mar 12 13:28:10 PDT 2019


Yep yep yep.

On Tue, Mar 12, 2019 at 1:24 PM Adrian Prantl via Phabricator
<reviews at reviews.llvm.org> wrote:
>
> aprantl added a comment.
>
> In D59235#1426716 <https://reviews.llvm.org/D59235#1426716>, @JDevlieghere wrote:
>
> > Agreed, and we've been doing this for new patches for a while now. However, I very strongly prefer having asserts over "returning a default value", which only hides real bugs.
>
>
> I think everyone is on the same page here, but it doesn't hurt to explicitly repeat this  :-)
>
> - Assertions should be used liberally to assert internal consistency and to enforce contracts in the API. Basically when you write an assertion, you should be already convinced that it will always hold.
> - Assertions may never be used to handle invalid external input in a parser. Invalid external input must use error handling, expected, optional, ...
>
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D59235/new/
>
> https://reviews.llvm.org/D59235
>
>
>


More information about the lldb-commits mailing list