Skip to content

clean-up: use Mutex reference p1#42670

Merged
botengyao merged 2 commits intoenvoyproxy:mainfrom
botengyao:clean_up
Dec 17, 2025
Merged

clean-up: use Mutex reference p1#42670
botengyao merged 2 commits intoenvoyproxy:mainfrom
botengyao:clean_up

Conversation

@botengyao
Copy link
Copy Markdown
Member

Replace absl::MutexLock::MutexLock(Mutex*) constructor with absl::MutexLock::MutexLock(Mutex&)
Similar to #41948

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: Boteng Yao <boteng@google.com>
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!

@botengyao botengyao enabled auto-merge (squash) December 17, 2025 21:14
@botengyao botengyao merged commit 692bc00 into envoyproxy:main Dec 17, 2025
24 checks passed
TAOXUY pushed a commit to TAOXUY/envoy that referenced this pull request Dec 18, 2025
Replace absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)
Similar to envoyproxy#41948

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: Xuyang Tao <taoxuy@google.com>
@botengyao botengyao deleted the clean_up branch December 30, 2025 19:45
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Mar 20, 2026
Replace absl::MutexLock::MutexLock(Mutex*) constructor with
absl::MutexLock::MutexLock(Mutex&)
Similar to envoyproxy#41948

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants