summaryrefslogtreecommitdiff
path: root/absl
diff options
context:
space:
mode:
Diffstat (limited to 'absl')
-rw-r--r--absl/algorithm/BUILD.bazel4
-rw-r--r--absl/algorithm/equal_benchmark.cc2
-rw-r--r--absl/base/BUILD.bazel4
-rw-r--r--absl/base/attributes.h3
-rw-r--r--absl/base/internal/direct_mmap.h2
-rw-r--r--absl/base/internal/thread_identity_benchmark.cc2
-rw-r--r--absl/container/BUILD.bazel8
-rw-r--r--absl/container/inlined_vector_benchmark.cc2
-rw-r--r--absl/debugging/failure_signal_handler_test.cc11
-rw-r--r--absl/strings/match.h3
-rw-r--r--absl/types/optional.h9
11 files changed, 34 insertions, 16 deletions
diff --git a/absl/algorithm/BUILD.bazel b/absl/algorithm/BUILD.bazel
index 5cd54960..3b24ce12 100644
--- a/absl/algorithm/BUILD.bazel
+++ b/absl/algorithm/BUILD.bazel
@@ -41,11 +41,11 @@ cc_test(
],
)
-cc_binary(
+cc_test(
name = "algorithm_benchmark",
- testonly = 1,
srcs = ["equal_benchmark.cc"],
copts = ABSL_TEST_COPTS,
+ tags = ["benchmark"],
deps = [
":algorithm",
"//absl/base:core_headers",
diff --git a/absl/algorithm/equal_benchmark.cc b/absl/algorithm/equal_benchmark.cc
index fea6d137..9af36ce7 100644
--- a/absl/algorithm/equal_benchmark.cc
+++ b/absl/algorithm/equal_benchmark.cc
@@ -15,8 +15,8 @@
#include <cstdint>
#include <cstring>
-#include "absl/algorithm/algorithm.h"
#include "benchmark/benchmark.h"
+#include "absl/algorithm/algorithm.h"
namespace {
diff --git a/absl/base/BUILD.bazel b/absl/base/BUILD.bazel
index bb7a5981..5c05c678 100644
--- a/absl/base/BUILD.bazel
+++ b/absl/base/BUILD.bazel
@@ -406,11 +406,11 @@ cc_test(
],
)
-cc_binary(
+cc_test(
name = "thread_identity_benchmark",
- testonly = 1,
srcs = ["internal/thread_identity_benchmark.cc"],
copts = ABSL_TEST_COPTS,
+ tags = ["benchmark"],
visibility = ["//visibility:private"],
deps = [
":base",
diff --git a/absl/base/attributes.h b/absl/base/attributes.h
index a4ec7e7c..b1883b6d 100644
--- a/absl/base/attributes.h
+++ b/absl/base/attributes.h
@@ -52,7 +52,8 @@
// Example:
//
// // Enable branches in the Abseil code that are tagged for ASan:
-// $ bazel -D ADDRESS_SANITIZER -fsanitize=address *target*
+// $ bazel build --copt=-DADDRESS_SANITIZER --copt=-fsanitize=address
+// --linkopt=-fsanitize=address *target*
//
// Since these macro names are only supported by GCC and Clang, we only check
// for `__GNUC__` (GCC or Clang) and the above macros.
diff --git a/absl/base/internal/direct_mmap.h b/absl/base/internal/direct_mmap.h
index 4bd273ed..2fe345fc 100644
--- a/absl/base/internal/direct_mmap.h
+++ b/absl/base/internal/direct_mmap.h
@@ -52,7 +52,7 @@
// SYS_mmap and SYS_munmap are not defined in Android.
#ifdef __BIONIC__
-extern "C" void* __mmap2(void*, size_t, int, int, int, long);
+extern "C" void* __mmap2(void*, size_t, int, int, int, size_t);
#if defined(__NR_mmap) && !defined(SYS_mmap)
#define SYS_mmap __NR_mmap
#endif
diff --git a/absl/base/internal/thread_identity_benchmark.cc b/absl/base/internal/thread_identity_benchmark.cc
index 3ae57317..fe22e4cf 100644
--- a/absl/base/internal/thread_identity_benchmark.cc
+++ b/absl/base/internal/thread_identity_benchmark.cc
@@ -12,10 +12,10 @@
// See the License for the specific language governing permissions and
// limitations under the License.
+#include "benchmark/benchmark.h"
#include "absl/base/internal/thread_identity.h"
#include "absl/synchronization/internal/create_thread_identity.h"
#include "absl/synchronization/internal/per_thread_sem.h"
-#include "benchmark/benchmark.h"
namespace {
diff --git a/absl/container/BUILD.bazel b/absl/container/BUILD.bazel
index 69cd5195..30341083 100644
--- a/absl/container/BUILD.bazel
+++ b/absl/container/BUILD.bazel
@@ -62,11 +62,11 @@ cc_test(
],
)
-cc_binary(
+cc_test(
name = "fixed_array_benchmark",
- testonly = 1,
srcs = ["fixed_array_benchmark.cc"],
copts = ABSL_TEST_COPTS + ["$(STACK_FRAME_UNLIMITED)"],
+ tags = ["benchmark"],
deps = [
":fixed_array",
"@com_github_google_benchmark//:benchmark",
@@ -117,11 +117,11 @@ cc_test(
],
)
-cc_binary(
+cc_test(
name = "inlined_vector_benchmark",
- testonly = 1,
srcs = ["inlined_vector_benchmark.cc"],
copts = ABSL_TEST_COPTS,
+ tags = ["benchmark"],
deps = [
":inlined_vector",
"//absl/base",
diff --git a/absl/container/inlined_vector_benchmark.cc b/absl/container/inlined_vector_benchmark.cc
index a2035e35..c6bc5cd9 100644
--- a/absl/container/inlined_vector_benchmark.cc
+++ b/absl/container/inlined_vector_benchmark.cc
@@ -17,9 +17,9 @@
#include <string>
#include <vector>
+#include "benchmark/benchmark.h"
#include "absl/base/internal/raw_logging.h"
#include "absl/strings/str_cat.h"
-#include "benchmark/benchmark.h"
namespace {
diff --git a/absl/debugging/failure_signal_handler_test.cc b/absl/debugging/failure_signal_handler_test.cc
index 33170dab..ba520910 100644
--- a/absl/debugging/failure_signal_handler_test.cc
+++ b/absl/debugging/failure_signal_handler_test.cc
@@ -132,8 +132,17 @@ constexpr int kFailureSignals[] = {
#endif
};
+std::string SignalParamToString(const ::testing::TestParamInfo<int>& info) {
+ std::string result = absl::debugging_internal::FailureSignalToString(info.param);
+ if (result.empty()) {
+ result = absl::StrCat(info.param);
+ }
+ return result;
+}
+
INSTANTIATE_TEST_CASE_P(AbslDeathTest, FailureSignalHandlerDeathTest,
- ::testing::ValuesIn(kFailureSignals));
+ ::testing::ValuesIn(kFailureSignals),
+ SignalParamToString);
#endif // GTEST_HAS_DEATH_TEST
diff --git a/absl/strings/match.h b/absl/strings/match.h
index 6005533c..108b6048 100644
--- a/absl/strings/match.h
+++ b/absl/strings/match.h
@@ -43,8 +43,7 @@ namespace absl {
//
// Returns whether a given std::string `haystack` contains the substring `needle`.
inline bool StrContains(absl::string_view haystack, absl::string_view needle) {
- return static_cast<absl::string_view::size_type>(haystack.find(needle, 0)) !=
- haystack.npos;
+ return haystack.find(needle, 0) != haystack.npos;
}
// StartsWith()
diff --git a/absl/types/optional.h b/absl/types/optional.h
index 42068cca..80a2d149 100644
--- a/absl/types/optional.h
+++ b/absl/types/optional.h
@@ -820,6 +820,12 @@ class optional : private optional_internal::optional_data<T>,
// only if `*this` is empty.
constexpr bool has_value() const noexcept { return this->engaged_; }
+// Suppress bogus warning on MSVC: MSVC complains call to reference() after
+// throw_bad_optional_access() is unreachable.
+#ifdef _MSC_VER
+#pragma warning(push)
+#pragma warning(disable : 4702)
+#endif // _MSC_VER
// optional::value()
//
// Returns a reference to an `optional`s underlying value. The constness
@@ -848,6 +854,9 @@ class optional : private optional_internal::optional_data<T>,
? reference()
: (optional_internal::throw_bad_optional_access(), reference()));
}
+#ifdef _MSC_VER
+#pragma warning(pop)
+#endif // _MSC_VER
// optional::value_or()
//