omjavaid abandoned this revision.
omjavaid added a comment.
Correction made upstream by @tberghammer.
http://reviews.llvm.org/D15355
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
This revision was automatically updated to reflect the committed changes.
Closed by commit rL255499: Add failure paths to a few JSONNumber members
(authored by omjavaid).
Changed prior to commit:
http://reviews.llvm.org/D15355?vs=42706=42712#toc
Repository:
rL LLVM
tberghammer added a comment.
I don't know what correction you are referring to as I haven't made any change
in JSON.{h,cpp} since you created this patch but I am happy with the current
situation (no warning on clang)
http://reviews.llvm.org/D15355
tberghammer requested changes to this revision.
tberghammer added a comment.
This revision now requires changes to proceed.
These GetAs{...} functions should never fail in their current implementation as
we have only 3 different data type and all of them are handled. The new
function signatures
labath added a subscriber: labath.
labath added a comment.
btw, if you use `llvm_unreachable("text")`, then you don't need the return
after that.
http://reviews.llvm.org/D15355
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
omjavaid created this revision.
omjavaid added a reviewer: tberghammer.
omjavaid added a subscriber: lldb-commits.
This patch updates GetAsUnsigned(), GetAsSigned(), and GetAsDouble() JSONNumber
functions to add failure check.
The previous code was generating compiler warnings for not being