[PATCH] D67213: [clang-tidy] Fix definitions in headers check to respect qualifiers

2019-09-05 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. thanks for the fix. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67213/new/ https://reviews.llvm.org/D67213 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D67213: [clang-tidy] Fix definitions in headers check to respect qualifiers

2019-09-05 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL371022: [clang-tidy] Fix definitions in headers check to respect qualifiers (authored by kadircet, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to c

[PATCH] D67213: [clang-tidy] Fix definitions in headers check to respect qualifiers

2019-09-05 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet created this revision. kadircet added a reviewer: gribozavr. Herald added subscribers: cfe-commits, xazax.hun. Herald added a project: clang. The check was generating a fix without taking qualifiers in return type into account. This patch changes the insertion location to be before qualif