kadircet added a comment. thanks, lgtm!
================ Comment at: clang-tools-extra/clangd/InlayHints.cpp:353 const HeuristicResolver *Resolver; PrintingPolicy TypeHintPolicy; + PrintingPolicy StructuredBindingPolicy; ---------------- can we document why we need to policies here, probably something like: ``` We want to suppress default template arguments and print canonical types, unfortunately they're conflicting policies hence we can't have both. For regular types, suppressing default template arguments is more important, whereas printing canonical types is crucial for structured bindings. Therefore we use 2 separate policies. ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108975/new/ https://reviews.llvm.org/D108975 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
