<div dir="ltr">This is a case where I think a new API *would* be warranted.  But it would not be an API specifically for the -m option.  It would instead be an API that introduces an `SBBreakpointOptions` class, and then add a method called `BreakpointCreateWithOptions(options)`.<div><br></div><div>Note that the general policy of the SB API is that you cannot delete or modify an existing method.  This kind of pattern provides an API that *can* be extended in the future, because you could, for example, add fields to SBBreakpointOptions and existing code wouldn't break, but new code could take advantage of the new options.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 31, 2017 at 8:32 AM Don Hinton <<a href="mailto:hintonda@gmail.com">hintonda@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Btw, is there a way to pass the '-m' option via the SB API?  I'd like to exclude matches in comments when using <span style="background-color:rgb(232,240,248);color:rgb(0,0,0)">BreakpointCreateBySourceRegex</span>.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 31, 2017 at 8:26 AM, Don Hinton <span dir="ltr"><<a href="mailto:hintonda@gmail.com" target="_blank">hintonda@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="m_-5617503736842056707h5">On Tue, Oct 31, 2017 at 8:22 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><br><div class="gmail_quote"><span><div dir="ltr">On Tue, Oct 31, 2017 at 8:12 AM Don Hinton <<a href="mailto:hintonda@gmail.com" target="_blank">hintonda@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>There have been a few suggestions that I could just use a script to solve this "problem" -- poor startup performance of clangdiag.</div><div><br></div><div>However, this patch was not submitted to solve a particular problem.  It was submitted in response to Jim's suggestion:</div><div><br></div><div><span style="color:rgb(80,0,80);font-size:12.8px">On Mon, Oct 23, 2017 at 6:25 PM, Jim Ingham </span><span dir="ltr" style="color:rgb(80,0,80);font-size:12.8px"><<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>></span><span style="color:rgb(80,0,80);font-size:12.8px"> wrote:</span><br style="color:rgb(80,0,80);font-size:12.8px"><blockquote class="gmail_quote" style="color:rgb(80,0,80);font-size:12.8px;margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Yeah, that would be easy to implement from the command line, maybe add a --file-is-regex flag or something.<br><br>From the SB API it would be better to have something like:<br><br>SBFileList SBTarget.GetFileListMatchingRegex("regex")<br><br>Please file an enhancement request for these of hack'em in if you're so motivated.<br></blockquote><div><br></div><div>clangdiag was only provided as a motivating example.</div></div></div></blockquote><div><br></div></span><div>Fair point, I had forgotten about that.  That said, I'm honestly still not that big of a fan.   It's no secret that I have a higher bar than others for options and API additions, but to me the potential use this particular option + SB API would get is not sufficient to justify the long term maintenance burden associated with having it.</div></div></div></blockquote><div><br></div></div></div><div>I'm not that familiar with the SB API part, but will investigate, and try to come up with a better motivation.</div><div><br></div><div>Thanks again for the suggestions and feedback...</div><div> </div></div><br></div></div>
</blockquote></div><br></div>
</blockquote></div>