<div dir="ltr">I don't recall, is there a hard restriction on debugserver not being allowed to use llvm code?  Because YAML is a superset of JSON</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 1, 2017 at 3:36 PM Davide Italiano via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also, FWIW, this is blatantly not the correct way of using assertions.<br>
It should either be assert(false && "patatino") or the more canonical<br>
llvm_unreachable("patatino").<br>
Relying on the implicit conversion from string to bool seems risky<br>
(and not really useful).<br>
<br>
On Fri, Dec 1, 2017 at 3:29 PM, Vedant Kumar via lldb-commits<br>
<<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
> Author: vedantk<br>
> Date: Fri Dec  1 15:29:04 2017<br>
> New Revision: 319596<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=319596&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=319596&view=rev</a><br>
> Log:<br>
> Fix warnings in JSON.cpp, NFC<br>
><br>
> These asserts are no-ops, and are supplanted by -Wcovered-switch.<br>
><br>
> Modified:<br>
>     lldb/trunk/tools/debugserver/source/JSON.cpp<br>
><br>
> Modified: lldb/trunk/tools/debugserver/source/JSON.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/JSON.cpp?rev=319596&r1=319595&r2=319596&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/JSON.cpp?rev=319596&r1=319595&r2=319596&view=diff</a><br>
> ==============================================================================<br>
> --- lldb/trunk/tools/debugserver/source/JSON.cpp (original)<br>
> +++ lldb/trunk/tools/debugserver/source/JSON.cpp Fri Dec  1 15:29:04 2017<br>
> @@ -58,7 +58,6 @@ uint64_t JSONNumber::GetAsUnsigned() con<br>
>    case DataType::Double:<br>
>      return (uint64_t)m_data.m_double;<br>
>    }<br>
> -  assert("Unhandled data type");<br>
>  }<br>
><br>
>  int64_t JSONNumber::GetAsSigned() const {<br>
> @@ -70,7 +69,6 @@ int64_t JSONNumber::GetAsSigned() const<br>
>    case DataType::Double:<br>
>      return (int64_t)m_data.m_double;<br>
>    }<br>
> -  assert("Unhandled data type");<br>
>  }<br>
><br>
>  double JSONNumber::GetAsDouble() const {<br>
> @@ -82,7 +80,6 @@ double JSONNumber::GetAsDouble() const {<br>
>    case DataType::Double:<br>
>      return m_data.m_double;<br>
>    }<br>
> -  assert("Unhandled data type");<br>
>  }<br>
><br>
>  void JSONNumber::Write(std::ostream &s) {<br>
><br>
><br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote></div>