couple of little spaces I missed...
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Nov 23 2022
Nov 23 2022
Remove whitespace changes on unaffected lines
Nov 22 2022
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
Nov 21 2022
Move getReplacementType to LLVM_16 block
Nov 19 2022
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
Nov 17 2022
Added release note, incorporated other review notes
Nov 11 2022
Nov 11 2022
Oct 19 2022
Oct 19 2022
Oct 18 2022
Oct 18 2022
Oct 14 2022
Oct 14 2022
Oct 13 2022
Oct 13 2022
Oct 10 2022
Oct 10 2022
anderslanglands added reviewers for D135557: Add needsImplicitDefaultConstructor and friends: tbaeder, aaron.ballman.
Fixed clang-format error
Oct 9 2022
Oct 9 2022
Sep 22 2022
Sep 22 2022
anderslanglands added a comment to D134416: Allow getting template args for ClassTemplateSpecializations.
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).
anderslanglands updated the diff for D134416: Allow getting template args for ClassTemplateSpecializations.
Add testing, address style notes
anderslanglands updated the diff for D134416: Allow getting template args for ClassTemplateSpecializations.
Updating doc comments to reflect new functionality
anderslanglands added reviewers for D134416: Allow getting template args for ClassTemplateSpecializations: tbaeder, aaron.ballman.
Hello again, this generalizes the template argument functionality to operate on cursor kinds other than functions.
anderslanglands requested review of D134416: Allow getting template args for ClassTemplateSpecializations.
Sep 16 2022
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 anderslangands@gmail.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
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