Crashpad: AnnotationList: Spell check a random word in annotation_list.h [crashpad/crashpad : main]

11 views
Skip to first unread message

Solomon Kinard (Gerrit)

unread,
Jul 10, 2025, 7:22:06 PMJul 10
to Mark Mentovai, crashp...@chromium.org
Attention needed from Mark Mentovai

Solomon Kinard added 1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Solomon Kinard . unresolved

Review? Reference [here](https://chromium-review.googlesource.com/c/chromium/src/+/6719473/comment/5cda037f_b12ec0d9). Carrying forward a question posed there:

What can be done systemically to prevent inadvertent fork changes of third party code? e.g. perhaps a presubmit warning for the uninitiated.What can be done systemically to prevent inadvertent fork changes of third party code? e.g. perhaps a presubmit warning for the uninitiated.

Open in Gerrit

Related details

Attention is currently required from:
  • Mark Mentovai
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
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: Ib7997b7753bba0e4e9a03daad08752e4a3e86b3e
Gerrit-Change-Number: 6725278
Gerrit-PatchSet: 1
Gerrit-Owner: Solomon Kinard <solomo...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Attention: Mark Mentovai <ma...@chromium.org>
Gerrit-Comment-Date: Thu, 10 Jul 2025 23:05:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
unsatisfied_requirement
open
diffy

Mark Mentovai (Gerrit)

unread,
Jul 10, 2025, 7:22:54 PMJul 10
to Solomon Kinard, crashp...@chromium.org
Attention needed from Solomon Kinard

Mark Mentovai voted and added 1 comment

Votes added by Mark Mentovai

Code-Review+1

1 comment

Patchset-level comments
Solomon Kinard . resolved

Review? Reference [here](https://chromium-review.googlesource.com/c/chromium/src/+/6719473/comment/5cda037f_b12ec0d9). Carrying forward a question posed there:

What can be done systemically to prevent inadvertent fork changes of third party code? e.g. perhaps a presubmit warning for the uninitiated.What can be done systemically to prevent inadvertent fork changes of third party code? e.g. perhaps a presubmit warning for the uninitiated.

Mark Mentovai

Review? Reference [here](https://chromium-review.googlesource.com/c/chromium/src/+/6719473/comment/5cda037f_b12ec0d9). Carrying forward a question posed there:

What can be done systemically to prevent inadvertent fork changes of third party code? e.g. perhaps a presubmit warning for the uninitiated.What can be done systemically to prevent inadvertent fork changes of third party code? e.g. perhaps a presubmit warning for the uninitiated.

The question is relevant to Chromium’s use of Crashpad, not Crashpad proper, so I’ll answer it there.

Open in Gerrit

Related details

Attention is currently required from:
  • Solomon Kinard
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: Ib7997b7753bba0e4e9a03daad08752e4a3e86b3e
Gerrit-Change-Number: 6725278
Gerrit-PatchSet: 1
Gerrit-Owner: Solomon Kinard <solomo...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Attention: Solomon Kinard <solomo...@chromium.org>
Gerrit-Comment-Date: Thu, 10 Jul 2025 23:22:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Solomon Kinard <solomo...@chromium.org>
satisfied_requirement
open
diffy

Solomon Kinard (Gerrit)

unread,
Jul 10, 2025, 7:24:23 PMJul 10
to Mark Mentovai, crashp...@chromium.org

Solomon Kinard voted and added 1 comment

Votes added by Solomon Kinard

Commit-Queue+2

1 comment

Patchset-level comments
Solomon Kinard . resolved

Thanks.

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: Ib7997b7753bba0e4e9a03daad08752e4a3e86b3e
Gerrit-Change-Number: 6725278
Gerrit-PatchSet: 1
Gerrit-Owner: Solomon Kinard <solomo...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Reviewer: Solomon Kinard <solomo...@chromium.org>
Gerrit-Comment-Date: Thu, 10 Jul 2025 23:24:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Solomon Kinard (Gerrit)

unread,
Jul 11, 2025, 11:47:45 PMJul 11
to Crashpad LUCI CQ, Mark Mentovai, crashp...@chromium.org

Solomon Kinard voted and added 1 comment

Votes added by Solomon Kinard

Commit-Queue+2

1 comment

Patchset-level comments
File-level comment, Patchset 2 (Latest):
Solomon Kinard . resolved

Thanks. This CL is now able to merge because I asked for crrev.com/c/6729599 to be created, which has also now been merged.

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: Ib7997b7753bba0e4e9a03daad08752e4a3e86b3e
Gerrit-Change-Number: 6725278
Gerrit-PatchSet: 2
Gerrit-Owner: Solomon Kinard <solomo...@chromium.org>
Gerrit-Reviewer: Mark Mentovai <ma...@chromium.org>
Gerrit-Reviewer: Solomon Kinard <solomo...@chromium.org>
Gerrit-Comment-Date: Sat, 12 Jul 2025 03:47:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Crashpad LUCI CQ (Gerrit)

unread,
Jul 11, 2025, 11:47:54 PMJul 11
to Solomon Kinard, Mark Mentovai, crashp...@chromium.org

Crashpad LUCI CQ submitted the change

Unreviewed changes

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Change information

Commit message:
Crashpad: AnnotationList: Spell check a random word in annotation_list.h
Bug: 415304336
Change-Id: Ib7997b7753bba0e4e9a03daad08752e4a3e86b3e
Commit-Queue: Solomon Kinard <solomo...@chromium.org>
Reviewed-by: Mark Mentovai <ma...@chromium.org>
Files:
  • M client/annotation_list.h
Change size: XS
Delta: 1 file changed, 1 insertion(+), 1 deletion(-)
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: Ib7997b7753bba0e4e9a03daad08752e4a3e86b3e
Gerrit-Change-Number: 6725278
Gerrit-PatchSet: 3
Gerrit-Owner: Solomon Kinard <solomo...@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