aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGravatar rsgowman <rgowman@google.com>2018-02-16 12:04:06 -0500
committerGravatar GitHub <noreply@github.com>2018-02-16 12:04:06 -0500
commitaa6f1ae0993ed948fb0b39283561bb3321eea5e9 (patch)
tree356c219ceac511926c2a440d9497f6f02b5dd2f7
parentef55eeff4288404ed92d7a3455005aa153dd1eaf (diff)
Disable -Wrange-loop-analysis for abseil (#807)
absl includes code like this: ``` void fn(std::initializer_list<absl::string_view> pieces) { ... for (const absl::string_view piece : pieces) total_size += piece.size(); ``` clang objects, suggesting that a reference should be used instead, i.e.: ``` for (const absl::string_view& piece : pieces) total_size += piece.size(); ``` But: a) we don't want to touch absl code b) string_views are cheap to copy (and absl recommends copying string_views rather than taking references as it may result in smaller code) c) some brief, naive benchmarking suggests there's no significant different in this case (i.e. (b) is correct.) Note that -Wrange-loop-analysis is already exlicitly enabled in our cmake build.
-rw-r--r--FirebaseFirestore.podspec2
1 files changed, 1 insertions, 1 deletions
diff --git a/FirebaseFirestore.podspec b/FirebaseFirestore.podspec
index 7af3b77..094a2ba 100644
--- a/FirebaseFirestore.podspec
+++ b/FirebaseFirestore.podspec
@@ -90,6 +90,6 @@ Google Cloud Firestore is a NoSQL document database built for automatic scaling,
]
ss.library = 'c++'
- ss.compiler_flags = '$(inherited) ' + '-Wno-comma'
+ ss.compiler_flags = '$(inherited) ' + '-Wno-comma -Wno-range-loop-analysis'
end
end