- User Since
- Mar 15 2017, 11:57 AM (90 w, 6 d)
Wed, Dec 5
Nov 5 2018
Oct 16 2018
Oct 15 2018
Oct 12 2018
Thanks for your input. I have opened https://reviews.llvm.org/D53220, which removes that option.
Oct 4 2018
I just tried this, this looks very promising! It should help build our outline view in a much more robust way than we do currently.
So I revisited this today. In the end, restarting clangd in our IDE works well. It should be merged anytime soon:
Sep 20 2018
Ohh awesome, I didn't know the LSP supported that. I'll try it it Theia when I have time.
Sep 13 2018
Sep 12 2018
Not sure who should review this, please feel free to add anybody who would be appropriate.
Sep 6 2018
Aug 29 2018
Aug 28 2018
Aug 27 2018
Aug 23 2018
Aug 10 2018
Aug 9 2018
Aug 8 2018
New version. I tried to share the code between this and SymbolCollector, but
didn't find a good clean way. Do you have concrete suggestions on how to do
this? Otherwise, would it be acceptable to merge it as-is?
Aug 6 2018
Aug 5 2018
Aug 3 2018
The tests now work on Windows, I added some path adjustment steps.
This was not tested on windows yet.
Aug 2 2018
That's fine with me. I'll try it and time will tell if I like it or not :).
Aug 1 2018
Add -Wno-nonportable-include-path to test/PCH/case-insensitive-include.c
@eric_niebler, question for you:
Jul 31 2018
Address Ilya's comment (add an indirection for initializationOptions).
Jul 26 2018
I think this addresses all of Ilya's comments.
Jul 25 2018
Thanks, that's simple and efficient. I'll update D49267 (to call reparseOpenFiles once again) once this is merged.
Fix tests on Windows
I uploaded a new version of this patch here:
Jul 24 2018
Jul 23 2018
Jul 17 2018
Jul 16 2018
Add tests for both == and !=.
- Add test
- Make operator== a function instead of method
- Add operator!= (so I can use EXPECT_NE in the test, and because it may be useful in general)
Jul 12 2018
Remove unintended changes
Note, D49265 in clang is a prerequisite.
- [clangd] JSONTracer: flush after writing event