From 01b471d9f3ebef27f5aaca14b66509099fa8cd6c Mon Sep 17 00:00:00 2001 From: Abseil Team Date: Wed, 26 Dec 2018 09:31:37 -0800 Subject: [PATCH] Export of internal Abseil changes. -- ffe1bf0e5f98c77cf4193f24ae9ce94d16a72c6e by Alex Strelnikov : Remove accidental duplication of conanfile.py. PiperOrigin-RevId: 226926125 -- daf639ddd32c57d1c5ab99b26a9b15107f47ce16 by Derek Mauro : Fix the CMake build for absl::container. This target is deprecated and will be removed in the future. Fixes https://github.com/abseil/abseil-cpp/pull/238 PiperOrigin-RevId: 226921798 -- b8ab2bb9081c266ced1d966c86d5b19af6b5b3ef by Abseil Team : Cleanup: Fix some ClangTidy warnings. PiperOrigin-RevId: 226678127 -- 8cdc95316fc8baba00073c38a444c089ed2d5f5e by Abseil Team : Cleanup: Fix some ClangTidy warnings. PiperOrigin-RevId: 226567814 GitOrigin-RevId: ffe1bf0e5f98c77cf4193f24ae9ce94d16a72c6e Change-Id: Idfe30b8b3229082eb7db4bfa928d3257be7dce1a --- CMake/README.md | 2 +- absl/conanfile.py | 83 ------------------------ absl/container/CMakeLists.txt | 4 -- absl/strings/escaping.cc | 4 +- absl/strings/internal/str_format/bind.cc | 4 +- absl/strings/strip_test.cc | 3 - 6 files changed, 5 insertions(+), 95 deletions(-) delete mode 100644 absl/conanfile.py diff --git a/CMake/README.md b/CMake/README.md index 79bbe24d5ad..66c3769296e 100644 --- a/CMake/README.md +++ b/CMake/README.md @@ -95,8 +95,8 @@ Here's a non-exhaustive list of Abseil CMake public targets: ```cmake absl::base absl::algorithm -absl::container absl::debugging +absl::flat_hash_map absl::memory absl::meta absl::numeric diff --git a/absl/conanfile.py b/absl/conanfile.py deleted file mode 100644 index d8307b4568e..00000000000 --- a/absl/conanfile.py +++ /dev/null @@ -1,83 +0,0 @@ -#!/usr/bin/env python -# -*- coding: utf-8 -*- - -# Note: Conan is supported on a best-effort basis. Abseil doesn't use Conan -# internally, so we won't know if it stops working. We may ask community -# members to help us debug any problems that arise. - -from conans import ConanFile, CMake, tools -from conans.errors import ConanInvalidConfiguration -from conans.model.version import Version - - -class AbseilConan(ConanFile): - name = "abseil" - url = "https://github.com/abseil/abseil-cpp" - homepage = url - author = "Abseil " - description = "Abseil Common Libraries (C++) from Google" - license = "Apache-2.0" - topics = ("conan", "abseil", "abseil-cpp", "google", "common-libraries") - exports = ["LICENSE"] - exports_sources = ["CMakeLists.txt", "CMake/*", "absl/*"] - generators = "cmake" - settings = "os", "arch", "compiler", "build_type" - - def configure(self): - if self.settings.os == "Windows" and \ - self.settings.compiler == "Visual Studio" and \ - Version(self.settings.compiler.version.value) < "14": - raise ConanInvalidConfiguration("Abseil does not support MSVC < 14") - - def build(self): - tools.replace_in_file("CMakeLists.txt", "project(absl)", "project(absl)\ninclude(conanbuildinfo.cmake)\nconan_basic_setup()") - cmake = CMake(self) - cmake.definitions["BUILD_TESTING"] = False - cmake.configure() - cmake.build() - - def package(self): - self.copy("LICENSE", dst="licenses") - self.copy("*.h", dst="include", src="absl") - self.copy("*.inc", dst="include", src="absl") - self.copy("*.a", dst="lib", src=".", keep_path=False) - self.copy("*.lib", dst="lib", src=".", keep_path=False) - - def package_info(self): - self.cpp_info.libs = ["absl_base", - "absl_synchronization", - "absl_strings", - "absl_symbolize", - "absl_malloc_internal", - "absl_time", - "absl_strings", - "absl_base", - "absl_dynamic_annotations", - "absl_spinlock_wait", - "absl_throw_delegate", - "absl_stacktrace", - "absl_int128", - "absl_span", - "test_instance_tracker_lib", - "absl_stack_consumption", - "absl_bad_any_cast", - "absl_hash", - "str_format_extension_internal", - "absl_failure_signal_handler", - "absl_str_format", - "absl_numeric", - "absl_any", - "absl_optional", - "absl_container", - "absl_debugging", - "absl_memory", - "absl_leak_check", - "absl_meta", - "absl_utility", - "str_format_internal", - "absl_variant", - "absl_examine_stack", - "absl_bad_optional_access", - "absl_algorithm"] - if self.settings.os == "Linux": - self.cpp_info.libs.append("pthread") diff --git a/absl/container/CMakeLists.txt b/absl/container/CMakeLists.txt index eb1b9ed72bf..4162d269ec2 100644 --- a/absl/container/CMakeLists.txt +++ b/absl/container/CMakeLists.txt @@ -20,10 +20,6 @@ absl_cc_library( NAME container - SRCS - "internal/raw_hash_set.cc" - COPTS - ${ABSL_DEFAULT_COPTS} PUBLIC ) diff --git a/absl/strings/escaping.cc b/absl/strings/escaping.cc index 8d8b00b2001..29f55c7c334 100644 --- a/absl/strings/escaping.cc +++ b/absl/strings/escaping.cc @@ -843,8 +843,8 @@ size_t Base64EscapeInternal(const unsigned char* src, size_t szsrc, char* dest, // Three bytes of data encodes to four characters of cyphertext. // So we can pump through three-byte chunks atomically. - if (szsrc >= 3) { // "limit_src - 3" is UB if szsrc < 3 - while (cur_src < limit_src - 3) { // as long as we have >= 32 bits + if (szsrc >= 3) { // "limit_src - 3" is UB if szsrc < 3. + while (cur_src < limit_src - 3) { // While we have >= 32 bits. uint32_t in = absl::big_endian::Load32(cur_src) >> 8; cur_dest[0] = base64[in >> 18]; diff --git a/absl/strings/internal/str_format/bind.cc b/absl/strings/internal/str_format/bind.cc index c4eddd17ddb..758adb7a256 100644 --- a/absl/strings/internal/str_format/bind.cc +++ b/absl/strings/internal/str_format/bind.cc @@ -25,12 +25,12 @@ class ArgContext { explicit ArgContext(absl::Span pack) : pack_(pack) {} // Fill 'bound' with the results of applying the context's argument pack - // to the specified 'props'. We synthesize a BoundConversion by + // to the specified 'unbound'. We synthesize a BoundConversion by // lining up a UnboundConversion with a user argument. We also // resolve any '*' specifiers for width and precision, so after // this call, 'bound' has all the information it needs to be formatted. // Returns false on failure. - bool Bind(const UnboundConversion *props, BoundConversion *bound); + bool Bind(const UnboundConversion* unbound, BoundConversion* bound); private: absl::Span pack_; diff --git a/absl/strings/strip_test.cc b/absl/strings/strip_test.cc index 40c4c607121..67355fcbcfe 100644 --- a/absl/strings/strip_test.cc +++ b/absl/strings/strip_test.cc @@ -27,9 +27,6 @@ namespace { -using testing::ElementsAre; -using testing::IsEmpty; - TEST(Strip, ConsumePrefixOneChar) { absl::string_view input("abc"); EXPECT_TRUE(absl::ConsumePrefix(&input, "a"));