Nathan Ridge
Nathan Ridge
Thanks, I understand the issue now, and can reproduce it as well.
I think the issue is in clang-tidy. When determining the location of the `)` insertion (for example [here](https://searchfox.org/llvm/rev/d338e535ec5f1de8b1b6cf7ea74514dfe1ecd0ce/clang-tools-extra/clang-tidy/bugprone/ImplicitWideningOfMultiplicationResultCheck.cpp#107)), the checker calls `FixItHint::CreateInsertion(E->getEndLoc(), ")")` (where `E` is the multiplication expression). `FixItHint::CreateInsertion()`...
Could clarify in which direction this goes wrong? * You are working in a C file, you want clangd to insert ``, but it inserts ``; or * You are...
Thanks. I tried this in a recent version of clangd, and it's working fine for me, i.e. it correctly inserts `` when editing a C++ file. To investigate further why...
Thanks for the additional information. It's possible the different behaviour we're seeing is due to being on different OSes (you're on Windows, I'm on Linux). Could you try the following...
I would also be curious to know what happens if you try completion with `std::uint64_t` rather than `uint64_t`. (I strongly suspect `` would be inserted in that case, but it...
Thanks for the additional details. Here's an outline of my understanding of how clangd determines the header to insert, using `uint64_t` / `std::uint64_t` as an example: 1. First, clangd checks...
Unfortunately, I'm not on Windows and therefore not in a good position to investigate further. Perhaps someone else who is on Windows can pick up this issue and find the...
> Documentation to llvm 8 [said](https://releases.llvm.org/8.0.0/tools/clang/tools/extra/docs/clangd/Installation.html) that to achieve what people ask in this bug, one needs to use clangd-indexer. These lines disappeared from llvm 10 documentation, it just says...
> Wait, Nathan? Eclipse CDT supporter, who suggested to use clangd instead of CDT's cpp-indexer? You actually work on clangd? :) I contribute to it, yeah :) I switched from...