[PATCH] D138377: add clang_Type_getFullyQualifiedName
Anders Langlands via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 22 23:16:15 PST 2022
anderslanglands added a comment.
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.
================
Comment at: clang/tools/c-index-test/c-index-test.c:1787
PrintCursor(cursor, NULL);
+ VisitorData *Data = (VisitorData *)d;
PrintSingleTypeSize(T, " [type=%s] [typekind=%s]", " [sizeof=%lld]",
----------------
aaron.ballman wrote:
> I'm not super familiar with c-index-test -- how certain are you that the client data will 1) be non-null, and 2) actually be a `VisitorData` pointer?
This was just copy-pasted from another function. I'm as certain as I can be given that all the tests pass :\
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138377/new/
https://reviews.llvm.org/D138377
More information about the cfe-commits
mailing list