aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJacob Bramley <jacob.bramley@arm.com>2019-10-02 15:40:23 +0100
committerJacob Bramley <jacob.bramley@arm.com>2019-10-02 15:40:23 +0100
commit32ceac2753283d5c8e865209b73ad87acd0d6a45 (patch)
tree5e28f65c32bf5c352f9aeaafb378eaed7a33590b
parent26384722e4b5859924cd1a7122efa0ffd621be4d (diff)
downloadvixl-32ceac2753283d5c8e865209b73ad87acd0d6a45.tar.gz
Disable clang-tidy mrand48 checks.
This should fix a CI failure, which for some reason only manifests on the MacOS bot. Change-Id: Ia526c8ef8b478710720c7a13be586646f021fe1a
-rw-r--r--.clang-tidy8
1 files changed, 7 insertions, 1 deletions
diff --git a/.clang-tidy b/.clang-tidy
index 5cb6253c..b3db0d43 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -1,6 +1,12 @@
---
# We use the clang-tidy defaults and the Google styles as a baseline, with a
# few changes specific to VIXL:
+# -clang-analyzer-security.insecureAPI.rand:
+# This warns against the use of mrand48 (etc) and suggests replacing them
+# with arc4random. However, we are using these to drive tests and debug
+# tools, and we need the ability to manually seed the generator. This is
+# not possible with arc4random, and do not particularly robust random
+# numbers, so we continue to use mrand48.
# -google-readability-todo:
# We don't put names on TODOs.
# -google-readability-function-size:
@@ -19,7 +25,7 @@
# google-readability-casting
#
# TODO: Also consider enabling other rules, such as bugprone-* and cert-*.
-Checks: 'google-*,-google-readability-todo,-google-readability-function-size,-google-build-using-namespace,-google-explicit-constructor,-google-readability-braces-around-statements,-google-readability-namespace-comments,-google-readability-casting'
+Checks: '-clang-analyzer-security.insecureAPI.rand,google-*,-google-readability-todo,-google-readability-function-size,-google-build-using-namespace,-google-explicit-constructor,-google-readability-braces-around-statements,-google-readability-namespace-comments,-google-readability-casting'
HeaderFilterRegex: '\.h$'
AnalyzeTemporaryDtors: false
CheckOptions: