Skip to content

Use mutex reference in lock constructors p3#41208

Merged
yanavlasov merged 1 commit intoenvoyproxy:mainfrom
yanavlasov:absl-mutex-lock
Sep 24, 2025
Merged

Use mutex reference in lock constructors p3#41208
yanavlasov merged 1 commit intoenvoyproxy:mainfrom
yanavlasov:absl-mutex-lock

Conversation

@yanavlasov
Copy link
Copy Markdown
Contributor

@yanavlasov yanavlasov commented Sep 24, 2025

Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with absl::MutexLock::MutexLock(Mutex&)

Risk Level: none
Testing: unit tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

@yanavlasov
Use mutex reference in lock constructors p3
af39bf9 
Signed-off-by: Yan Avlasov <yavlasov@google.com>
@yanavlasov yanavlasov enabled auto-merge (squash) September 24, 2025 16:16
adisuissa
adisuissa approved these changes Sep 24, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@adisuissa adisuissa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@yanavlasov yanavlasov merged commit 572ec5b into envoyproxy:main Sep 24, 2025
25 checks passed
@yanavlasov yanavlasov deleted the absl-mutex-lock branch September 24, 2025 20:07
This was referenced Nov 10, 2025
Merged
Merged
Merged
Merged
Merged
KBaichoo pushed a commit that referenced this pull request Nov 11, 2025
@adisuissa
Use mutex reference in lock constructors p8 (#41948)
eb6c7d4 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to #41208.

Risk Level: low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
KBaichoo pushed a commit that referenced this pull request Nov 11, 2025
@adisuissa
Use mutex reference in lock constructors p4 (#41938)
18ae3a3 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to #41208.

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
KBaichoo pushed a commit that referenced this pull request Nov 11, 2025
@adisuissa
Use mutex reference in lock constructors p5 (#41939)
0c2829d 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to #41208.

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
KBaichoo pushed a commit that referenced this pull request Nov 11, 2025
@adisuissa
Use mutex reference in lock constructors p6 (#41943)
2bdab3a 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to #41208.

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
This was referenced Dec 17, 2025
Merged
Merged
botengyao added a commit that referenced this pull request Dec 17, 2025
@botengyao
clean-up: use mutex reference p3 (#42674)
9b74448 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Similar to #41208.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:

Signed-off-by: Boteng Yao <boteng@google.com>
TAOXUY pushed a commit to TAOXUY/envoy that referenced this pull request Dec 18, 2025
@botengyao @TAOXUY
clean-up: use mutex reference p3 (envoyproxy#42674)
4a97d68 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Similar to envoyproxy#41208.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:

Signed-off-by: Boteng Yao <boteng@google.com>
Signed-off-by: Xuyang Tao <taoxuy@google.com>
This was referenced Dec 18, 2025
Merged
Merged
botengyao added a commit that referenced this pull request Dec 18, 2025
@botengyao
clean-up: use mutex reference p4 (#42675)
8035529 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Similar to #41208.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

---------

Signed-off-by: Boteng Yao <boteng@google.com>
@adisuissa adisuissa mentioned this pull request Dec 18, 2025
Merged
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
@adisuissa @grnmeira
Use mutex reference in lock constructors p8 (envoyproxy#41948)
ebb8f52 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to envoyproxy#41208.

Risk Level: low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
Signed-off-by: Gustavo <grnmeira@gmail.com>
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
@adisuissa @grnmeira
Use mutex reference in lock constructors p4 (envoyproxy#41938)
b029752 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to envoyproxy#41208.

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
Signed-off-by: Gustavo <grnmeira@gmail.com>
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
@adisuissa @grnmeira
Use mutex reference in lock constructors p5 (envoyproxy#41939)
a1ae052 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to envoyproxy#41208.

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
Signed-off-by: Gustavo <grnmeira@gmail.com>
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
@adisuissa @grnmeira
Use mutex reference in lock constructors p6 (envoyproxy#41943)
1d43930 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Additional Description: needed due to upcoming absl change.
Similar to envoyproxy#41208.

Signed-off-by: Adi Suissa-Peleg <adip@google.com>
Signed-off-by: Gustavo <grnmeira@gmail.com>
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
@botengyao @grnmeira
clean-up: use mutex reference p3 (envoyproxy#42674)
a4a6474 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Similar to envoyproxy#41208.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:

Signed-off-by: Boteng Yao <boteng@google.com>
Signed-off-by: Gustavo <grnmeira@gmail.com>
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
@botengyao @grnmeira
clean-up: use mutex reference p4 (envoyproxy#42675)
0ed93f6 
Replace deprecated absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)

Similar to envoyproxy#41208.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

---------

Signed-off-by: Boteng Yao <boteng@google.com>
Signed-off-by: Gustavo <grnmeira@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adisuissa adisuissa adisuissa approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yanavlasov @adisuissa