couple of little spaces I missed...
- All Stories
- Advanced Search
- Transaction Logs
Nov 23 2022
Remove whitespace changes on unaffected lines
Nov 22 2022
Re the whitespace - yeah that's me running clang-format on the whole function rather than just the lines I changed. I'll see if I can split those out.
Now tested by adding a -print-qualified-type-names flag to c-index-test and creating a (small) dedicated test in print-qualified-type.cpp
Nov 21 2022
Move getReplacementType to LLVM_16 block
Nov 19 2022
Arg just noticed an issue: clang_Type_getReplacementType should be in the LLVM_16 block in libclang.map not in the LLVM_13 block. I've blown away my local branch so don't know how to fix the patch. Shall I just make a new patch once this is merged?
Great, thanks! I've got a couple more things to land once this is merged and I've cleaned them up. Do you wanna show me how to do the merge myself?
Nov 17 2022
Added release note, incorporated other review notes
Nov 11 2022
Oct 19 2022
Oct 18 2022
Oct 14 2022
Oct 13 2022
Oct 10 2022
Fixed clang-format error
Oct 9 2022
Sep 22 2022
I ended up modifying c-index-test to print out the template arguments in the same way it does for FunctionDecl, which ends up working all cases in the existing test suite. The only other thing I added was a specific case in index-templates.cpp to check whether it catches StructDecl too (it does).
Add testing, address style notes
Updating doc comments to reflect new functionality
Hello again, this generalizes the template argument functionality to operate on cursor kinds other than functions.
Sep 16 2022
Yes it's probably safest if someone commits on my behalf, unless someone's got time to walk me through it. Just use Anders Langlands email@example.com for attribution please.
Adding missing backtick
OK that should all be good now. Thanks for your patience, it's my first time doing all this.
Adding in the dyn_cast_is_present change I missed on the previous one
squashing all commits into one to hopefully fix the revision
Sep 15 2022
Sorry, this was supposed to update https://reviews.llvm.org/D13392. How do I delete this?
Updating based on review
- Add new function to release notes
- Fix failing test and add a new test specifically for this function