<div dir="auto"><div>It also doesn't seem to me that the benefits outweigh the costs. (Arguments below). Not strongly opposed if others like it, but please make it C++ rather than python. (Happy to elaborate if that's controversial).</div><div dir="auto"><div dir="auto"><br></div><div dir="auto">The divergence in code is small and isolated, and the production code *is* covered by lit tests. This change to the tests seems unlikely to catch any actual bugs. It seems just as likely we'll have bugs in the transcoder.</div><div dir="auto"><br></div><div dir="auto">In addition to a more maintainable input format, -lit-test triggers a couple more behaviors that are important for testing but not particularly relevant to any particular test. One of the (IMO) antipatterns of lit tests is they accumulate flags and noise that aren't particularly related to the current test and obscure intent. Splitting out the transcoder would make it harder to bundle our generic test behavior this way, as well as adding noise to each test for pipeline mechanics.</div><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Tue, Aug 14, 2018, 14:52 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" rel="noreferrer">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" rel="noreferrer">clangd-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev" rel="noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev</a><br>
</blockquote></div></div></div>