Skip to content

feature: support more than 64 native attributes#293

Merged
KishanPRao merged 2 commits into
masterfrom
feature/crashpad-unbounded-attributes
Jul 7, 2026
Merged

feature: support more than 64 native attributes#293
KishanPRao merged 2 commits into
masterfrom
feature/crashpad-unbounded-attributes

Conversation

@KishanPRao

@KishanPRao KishanPRao commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes the 64-entry limit on native (Crashpad) attributes.

Previously, a simple dictionary was used for storing attributes, which limits the total count to 64 entries. They are now stored in Crashpad's global AnnotationList.

The attribute count is limited by Crashpad's internal limit, kMaxNumberOfAnnotations. SetAnnotation will refuse to add keys beyond the limit, providing feedback with a warning.

Changes

  • Updates Backtrace & Crashpad libraries for Windows & Android.
  • Native crash reports omit attributes with an empty-string value (this was allowed previously).

ref: BT-6868

@KishanPRao
KishanPRao requested a review from melekr June 16, 2026 13:41
@KishanPRao KishanPRao self-assigned this Jun 16, 2026
@KishanPRao

KishanPRao commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author
  • Wait for update libraries

@KishanPRao KishanPRao changed the title feature: allow addition of unbounded count of attributes feature: support more than 64 native attributes Jul 3, 2026

@melekr melekr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@KishanPRao
KishanPRao marked this pull request as ready for review July 7, 2026 17:24

@melekr melekr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

send it 🚀

@KishanPRao
KishanPRao merged commit 3d547c7 into master Jul 7, 2026
18 of 20 checks passed
@KishanPRao KishanPRao mentioned this pull request Jul 9, 2026
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.

3 participants