<div dir="ltr">Sorry for not getting to the review, got distracted by other things.<div><br></div><div>I'll be on vacation till Jul 23, so I won't be available before that date. However, AFAIK Sam is still planning to take a look at the changes and I'm happy to help with it after getting back.</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 28, 2018 at 9:27 PM Sam McCall 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 dir="auto">Also happy to take a look here.<div dir="auto">I think the split you described is right - the details of XPC are more interesting to Apple, the interfaces are of wider interest.</div><div dir="auto">I'd still like to look at least a bit at each patch, to understand how we should be adapting if the JSON output and/or protocol structs change in future and so on.</div><div dir="auto"><br></div><div dir="auto">Will try to get into this tomorrow, thanks for the heads-up!</div><div dir="auto"><br></div><div dir="auto">Cheers, Sam</div><br><br><div class="gmail_quote" dir="auto"><div dir="ltr">On Thu, Jun 28, 2018, 20:07 Jan Korous via clangd-dev <<a href="mailto:clangd-dev@lists.llvm.org" target="_blank">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 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" 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" 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" 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" target="_blank">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>
_______________________________________________<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>