<div dir="ltr">Thanks for the reply! I'll work on the patch to add the support for fix-its.<div class="gmail_extra"><br><div class="gmail_quote">On 2 August 2018 at 03:11, Ilya Biryukov <span dir="ltr"><<a href="mailto:ibiryukov@google.com" target="_blank">ibiryukov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Alex,<div><br></div><div>We discussed it internally inside our team and I think everyone agrees that would be the right thing to do. We don't work on it currently, though, so no overlap on our side.</div><div>Unfortunately, LSP does not support this, so this would be an extension.</div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>BTW, consider implementing the current scheme for fix-its that uses textDocument/codeAction and matches on sent diagnostics. It's not ideal, as it forces clangd to stash fix-its in memory. But it does work well in practice (we've tested only in VSCode, though).</div><div>If you don't need to support applying diagnostics in batches, this approach might work well for your use-case too.</div></div></blockquote><div><br></div><div>Unfortunately the existing scheme doesn't really work with our system. We'd like to pass in all of the data we need in publishDiagnostics, and let the client apply the fix-its.</div><div><br></div><div>Cheers,</div><div>Alex</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div><br></div><div><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Wed, Aug 1, 2018 at 10:19 PM Alex L via clangd-dev <<a href="mailto:clangd-dev@lists.llvm.org" target="_blank">clangd-dev@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Hi,<div><br></div><div>Has there been any discussion on sending fixits from the server to client as part of the 'publishDiagnostics' LSP response?</div><div>I would like to start working on an extension for this (disabled by default), and would like to avoid duplicate work if there are already people working on it :)</div><div><br></div><div>Thanks,</div><div>Alex</div></div></div></div>
______________________________<wbr>_________________<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/<wbr>mailman/listinfo/clangd-dev</a><span class="HOEnZb"><font color="#888888"><br>
</font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br clear="all"><div><br></div>-- <br><div dir="ltr" class="m_7880654777952403616gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div></font></span></div></div>
</blockquote></div><br></div></div>