<div dir="auto"><div dir="auto">Hi Jan,</div><div dir="auto"><br></div><div dir="auto">Happy to take a look at the changes in the common code.</div><div dir="auto">Will get to it as soon as I have some spare time. (Hopefully tomorrow or on Monday)</div></div><br><div class="gmail_quote"><div dir="ltr">Jan Korous via clangd-dev <<a href="mailto:clangd-dev@lists.llvm.org" target="_blank" rel="noreferrer">clangd-dev@lists.llvm.org</a>> schrieb am Do., 28. Juni 2018, 20:07:<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 all,<div><br></div><div>I would like to ask if there are volunteers to help us with code review - we finally have patches with alternative transport layer we promised at EuroLLVM ready.</div><div><br></div><div>While I assume (but please correct me if I am wrong) that it would be fine to have our specific XPC stuff reviewed by other Apple folks I feel it would be more appropriate to have changes impacting the common code reviewed by clangd community.</div><div><br></div><div>If anyone is willing to take a look - patches are here: </div><div><br></div><div><div>[clangd] refactoring for XPC transport layer [NFCI]</div><div><a href="https://reviews.llvm.org/D48559" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D48559</a></div></div><div>This is probably the most interesting patch for the community as it changes current clangd design in order to be able to accommodate alternative dispatcher.</div><div><br></div><div><div><div>[clangd] JSON <-> XPC conversions</div></div></div><div><a href="https://reviews.llvm.org/D48560" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D48560</a></div><div>Small, self-contained patch - conversion functions and tests.</div><div><br></div><div><div>[clangd] XPC transport layer</div></div><div><a href="https://reviews.llvm.org/D48562" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D48562</a></div><div>Implementation of alternative transport layer. We are currently having this patch reviewed internally but if there’s interest in community it would be awesome to have it reviewed on phab as well.</div><div><br></div><div>Thank you.</div><div><br></div><div>Jan</div></div>_______________________________________________<br>
clangd-dev mailing list<br>
<a href="mailto:clangd-dev@lists.llvm.org" rel="noreferrer noreferrer" target="_blank">clangd-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev</a><br>
</blockquote></div>