return a bool from LogOutputStream::Delegate::Log [crashpad/crashpad : main]

4 views
Skip to first unread message

Joshua Peraza (Gerrit)

unread,
Jul 24, 2025, 8:55:09 PMJul 24
to Mark Mentovai, Crashpad LUCI CQ, crashp...@chromium.org
Attention needed from Mark Mentovai

Joshua Peraza voted Commit-Queue+1

Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Mark Mentovai
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I3f78d25ca773bb3d580612454dded002084095db
Gerrit-Change-Number: 6786151
Gerrit-PatchSet: 3
Gerrit-Owner: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Attention: Mark Mentovai <ma...@chromium.org>
Gerrit-Comment-Date: Fri, 25 Jul 2025 00:55:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Mark Mentovai (Gerrit)

unread,
Jul 24, 2025, 10:10:13 PMJul 24
to Joshua Peraza, Crashpad LUCI CQ, crashp...@chromium.org
Attention needed from Joshua Peraza

Mark Mentovai voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Joshua Peraza
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I3f78d25ca773bb3d580612454dded002084095db
Gerrit-Change-Number: 6786151
Gerrit-PatchSet: 3
Gerrit-Owner: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Attention: Joshua Peraza <jpe...@chromium.org>
Gerrit-Comment-Date: Fri, 25 Jul 2025 02:10:10 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Joshua Peraza (Gerrit)

unread,
Jul 24, 2025, 10:12:51 PMJul 24
to Mark Mentovai, Crashpad LUCI CQ, crashp...@chromium.org

Joshua Peraza voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I3f78d25ca773bb3d580612454dded002084095db
Gerrit-Change-Number: 6786151
Gerrit-PatchSet: 3
Gerrit-Owner: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Comment-Date: Fri, 25 Jul 2025 02:12:48 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Crashpad LUCI CQ (Gerrit)

unread,
Jul 24, 2025, 10:13:00 PMJul 24
to Joshua Peraza, Mark Mentovai, crashp...@chromium.org

Crashpad LUCI CQ submitted the change

Change information

Commit message:
return a bool from LogOutputStream::Delegate::Log

__android_log_buf_write() has been updated to no longer return
-EAGAIN. It now returns 1 on success, otherwise -EPERM.
Change-Id: I3f78d25ca773bb3d580612454dded002084095db
Commit-Queue: Joshua Peraza <jpe...@chromium.org>
Reviewed-by: Mark Mentovai <ma...@chromium.org>
Files:
  • M handler/linux/crash_report_exception_handler.cc
  • M util/stream/log_output_stream.cc
  • M util/stream/log_output_stream.h
  • M util/stream/log_output_stream_test.cc
Change size: S
Delta: 4 files changed, 10 insertions(+), 14 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Mark Mentovai
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: crashpad/crashpad
Gerrit-Branch: main
Gerrit-Change-Id: I3f78d25ca773bb3d580612454dded002084095db
Gerrit-Change-Number: 6786151
Gerrit-PatchSet: 4
Gerrit-Owner: Joshua Peraza <jpe...@chromium.org>
Gerrit-Reviewer: Crashpad LUCI CQ <crashpa...@luci-project-accounts.iam.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages