<div dir="ltr">This seems reasonable to me.  It's worth pointing out though that in regards to the last comment "<span style="color:rgb(33,33,33)">IMO it's good to make this lldb-vscode more general so that it can be used by other debugger frontends besides vscode", despite the name lldb-vscode, there is actually nothing here that is specific to VSCode.  It reads DAP requests on stdin and responds with DAP responses on stdout.  That's literally it.  The only thing vscode specific about it is the names of the source files and some internal classes.  I actually wouldn't be opposed to changing it to lldb-dap</span></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 12, 2019 at 12:34 PM Leonard Mosescu via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@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"><div dir="ltr">Greg, what do you think?<div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 12, 2019 at 11:50 AM Qianli Ma <<a href="mailto:chienlima@google.com" target="_blank">chienlima@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi lldb community,<div><br></div><div>I am currently working on a project related to lldb. I'd like to write a DAP RPC server similars to<a href="https://github.com/llvm-mirror/lldb/blob/master/tools/lldb-vscode/lldb-vscode.cpp" target="_blank"> lldb-vscode.cc</a> but exports I/O to internal RPC clients. Doing so requires me to reuse some functions defined in <a href="https://github.com/llvm-mirror/lldb/blob/master/tools/lldb-vscode/lldb-vscode.cpp" target="_blank">lldb-vscode.cc</a>. However as those functions are defined using forward declaration I am not able to do that.</div><div><br></div><div>I'd like refactor the code a bit. More specifically, I'd like to extract all helper functions in <a href="https://github.com/llvm-mirror/lldb/blob/master/tools/lldb-vscode/lldb-vscode.cpp" target="_blank">lldb-vscode.cc</a> into a separate file and create a header for it.  BTW, IMO it's good to make this lldb-vscode more general so that it can be used by other debugger frontends besides vscode.</div><div><br></div><div>Please let me know WDYT and how I can proceed to submit changes for review. </div><div><br></div><div>Thanks and Regards</div><div>Qianli</div></div>
</blockquote></div>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
</blockquote></div>