Skip to content

Fix abseil deprecations (original by alona.moony@gmail.com) - #666

Merged
adonovan merged 1 commit into
google:mainfrom
dneto0:pr644
Oct 2, 2026
Merged

adonovan merged 1 commit into
google:mainfrom
dneto0:pr644

Conversation

@dneto0

@dneto0 dneto0 commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

The original work is by Alona Enraght-Moony alona.moony@gmail.com but I'm signing this so it can land.

It was PR #644 and the author signed the Google CLA (verified by the CLA-bot).

Author: Alona Enraght-Moony alona.moony@gmail.com
Date: Mon Jun 29 23:08:33 2026 +0100

Fix abseil lock deprecations

Fixes https://github.com/google/re2/issues/631

These were deprecated in
https://github.com/abseil/abseil-cpp/commit/6d8e1a5cf00d6a6b26069b0baf0dd9b855a17459

The original work is by Alona Enraght-Moony <alona.moony@gmail.com>
but I'm signing this so it can land.

It was PR google#644 and the author
signed the Google CLA (verified by the CLA-bot).

Author: Alona Enraght-Moony <alona.moony@gmail.com>
Date:   Mon Jun 29 23:08:33 2026 +0100

    Fix abseil lock deprecations

    Fixes google#631

    These were deprecated in
    abseil/abseil-cpp@6d8e1a5

@dneto0 dneto0 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also retested with a recent copy of abseil-cpp
It passed all the tests I could find:

./re2/regexp_test
./re2/exhaustive3_test
./re2/charclass_test
./re2/filtered_re2_test
./re2/search_test
./re2/random_test
./re2/possible_match_test
./re2/mimics_pcre_test
./re2/set_test
./re2/parse_test
./re2/re2_test
./re2/compile_test
./re2/dfa_test
./re2/exhaustive1_test
./re2/re2_arg_test
./re2/exhaustive_test
./re2/exhaustive2_test
./re2/simplify_test
./re2/required_prefix_test
./re2/string_generator_test

@dneto0

dneto0 commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

cc: @adonovan

@adonovan
adonovan merged commit 2da0056 into google:main Oct 2, 2026
35 checks passed
@adonovan

adonovan commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

Many thanks for pushing this over the hump.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants