<div dir="ltr">Sounds like we don't have a test for thread backtrace -s.  I know I usually argue against tests of the CLI, but only when they're used instead of tests for the api.  Seems like we should have at least one test for every option of every command.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Feb 5, 2016 at 4:35 PM Jim Ingham 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">Author: jingham<br>
Date: Fri Feb  5 18:31:23 2016<br>
New Revision: 259962<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=259962&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=259962&view=rev</a><br>
Log:<br>
Fix "thread backtrace -s": option was misparsed because of a missing break.<br>
<br>
<rdar://problem/24525106><br>
<br>
Modified:<br>
    lldb/trunk/source/Commands/CommandObjectThread.cpp<br>
<br>
Modified: lldb/trunk/source/Commands/CommandObjectThread.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectThread.cpp?rev=259962&r1=259961&r2=259962&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectThread.cpp?rev=259962&r1=259961&r2=259962&view=diff</a><br>
==============================================================================<br>
--- lldb/trunk/source/Commands/CommandObjectThread.cpp (original)<br>
+++ lldb/trunk/source/Commands/CommandObjectThread.cpp Fri Feb  5 18:31:23 2016<br>
@@ -194,6 +194,7 @@ public:<br>
                     if (!success)<br>
                         error.SetErrorStringWithFormat("invalid integer value for option '%c'", short_option);<br>
                 }<br>
+                break;<br>
                 case 'e':<br>
                 {<br>
                     bool success;<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>
</blockquote></div>