[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