[PATCH] D40325: add new check to find OSSpinlock usage
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 22 11:36:59 PST 2017
hokein added inline comments.
================
Comment at: clang-tidy/objc/AvoidSpinlockCheck.cpp:35
+ diag(MatchedExpr->getLocStart(),
+ "OSSpinlock is deprecated on iOS. Please use other locks or dispatch "
+ "queue.");
----------------
hokein wrote:
> nit: clang-tidy message is not a complete message, please follow the rule.
>
> The `OSSpinlock` is also deprecated on macOS, I think? from https://developer.apple.com/documentation/os/1646466-os_unfair_lock_lock.
oops, sorry for my typo: s/is not a complete message/is not a complete sentence.
https://reviews.llvm.org/D40325
More information about the cfe-commits
mailing list