<div dir="ltr">Hi Jan,<div><br></div><div>I agree that '-input-style=delimited' and '-lit-test' does look like something that shouldn't be in clangd from the architectural standpoint, as it's only specific to the test.<br></div><div>In practice, the python wrapper will probably be more complicated than what we have now and could even make some things harder, e.g. it's very easy to debug failures now and I could imagine problems with debugging a python wrapper.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 14, 2018 at 2:52 PM Jan Korous via clangd-dev <<a href="mailto:clangd-dev@lists.llvm.org">clangd-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 style="word-wrap:break-word;line-break:after-white-space">Hi,<div><br></div><div>We were discussing <a href="https://reviews.llvm.org/D50641" target="_blank">https://reviews.llvm.org/D50641</a> over lunch and an idea to replace -lit-test by an utility appeared (shoutout to Dan Liew!).</div><div><br></div><div>I don’t particularly like to have specific code path for testing but I see the value it provides. How about I implement a small python script or a binary that would read JSON strings separated by triple-dash from stdin and write proper LSP messages to stdout (with LSP headers prepended)?</div><div><br></div><div>What do you think?</div><div><br></div><div>Cheers,</div><div><br></div><div>Jan</div><div><br></div><div><br></div></div>_______________________________________________<br>
clangd-dev mailing list<br>
<a href="mailto:clangd-dev@lists.llvm.org" target="_blank">clangd-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div>