<div><div dir="auto">+jingham</div><div dir="auto"><br></div><br><div class="gmail_quote"><div>On Fri, Oct 20, 2017 at 6:57 AM Don Hinton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">hintonda added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D36347#901885" rel="noreferrer" target="_blank">https://reviews.llvm.org/D36347#901885</a>, @zturner wrote:<br>
<br>
> One possible reason for why this never got any traction is that `lldb-commits` wasn't added as a subscriber.  While it's true that the tagged people should have chimed in, having the whole commits list will get some more visibility.  I never saw this come to my inbox.<br>
><br>
> I think this would be most suitable in the `lldb/examples` folder.<br>
><br>
> I can't really review this thoroughly, because it relies on a bash script, and I use Windows where we bash isn't really a thing.  My bash is rusty, but it looks like you're embedding a python script in the bash script?  It might be good if this were just an lldb script command.  Take a look at `command script add` in LLDB, and in the `examples` folder for some examples of existing commands that work this way.  The nice thing about doing it this way is that you could just be inside LLDB and write `(lldb) break-diag -Wcovered-switch`, for example, which would be a much tighter integration with the debugger.<br>
<br>
<br>
Thanks for taking a look.<br>
<br>
I mainly work on *nix systems, hence the initial shell script, but if there's an interest, I'll be happy to convert it to a single python script as you suggest, and resubmit it as a patch to lldb.<br>
<br>
Thanks again...<br>
<br>
<br>
<a href="https://reviews.llvm.org/D36347" rel="noreferrer" target="_blank">https://reviews.llvm.org/D36347</a><br>
<br>
<br>
<br>
</blockquote></div></div>