fmayer added inline comments.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/android-comparison-in-temp-failure-retry.c:1 -// RUN: %check_clang_tidy %s android-comparison-in-temp-failure-retry %t +// RUN: %check_clang_tidy %s android-comparison-in-temp-failure-retry %t -- -config="{CheckOptions: [{key: android-comparison-in-temp-failure-retry.RetryMacros, value: 'TEMP_FAILURE_RETRY,MY_TEMP_FAILURE_RETRY'}]}" ---------------- srhines wrote: > Ideally there should either be another RUN macro here without your new > options, or there should be a separate test/file for checking here, since > we'd like to make sure that the default behavior still works. Left this file alone, and added a new file for custom macros. ================ Comment at: clang-tools-extra/test/clang-tidy/checkers/android-comparison-in-temp-failure-retry.c:28 TEMP_FAILURE_RETRY(foo()); TEMP_FAILURE_RETRY((foo())); ---------------- srhines wrote: > Don't you want a check for `MY_TEMP_FAILURE_RETRY` for cases like this too > (to make sure that it is detecting them)? Checking for more cases in the other file. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83144/new/ https://reviews.llvm.org/D83144 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits