Skip to content

chore: Add [[nodiscard]] attribute to native functions#660

Merged
msluszniak merged 5 commits intomainfrom
@ms/add-nodiscard-attr
Jan 16, 2026
Merged

chore: Add [[nodiscard]] attribute to native functions#660
msluszniak merged 5 commits intomainfrom
@ms/add-nodiscard-attr

Conversation

@mkopcins
Copy link
Copy Markdown
Collaborator

Description

As in the title, add [[nodiscard]] attribute to exported functions in native implementation.

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

Check if all demo apps work the same as previously

Screenshots

Related issues

Closes #448

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@msluszniak msluszniak changed the title chore: Add [[nodiscard]] attribute to native functions chore: Add [[nodiscard]] attribute to native functions Oct 23, 2025
@msluszniak msluszniak self-assigned this Oct 23, 2025
@msluszniak msluszniak added the chore PRs that are chores label Oct 23, 2025
@msluszniak msluszniak requested a review from IgorSwat October 23, 2025 12:25
@msluszniak msluszniak requested a review from chmjkb January 15, 2026 14:49
@chmjkb
Copy link
Copy Markdown
Collaborator

chmjkb commented Jan 16, 2026

Can you resolve conflicts before I review this? cc @msluszniak

@msluszniak
Copy link
Copy Markdown
Member

@chmjkb sure thing, the conflicts comes from today's PRs ;)

@msluszniak
Copy link
Copy Markdown
Member

@chmjkb conflicts resolved

## Description

Hotfix for compilation error. Readied +operator 

### Introduces a breaking change?

- [x] Yes
- [ ] No

### Type of change

- [x] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [ ] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

### Tested on

- [x] iOS
- [x] Android
@msluszniak msluszniak merged commit 8800e00 into main Jan 16, 2026
4 of 5 checks passed
@msluszniak msluszniak deleted the @ms/add-nodiscard-attr branch January 16, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore PRs that are chores

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mark native functions with [[nodiscard]]

4 participants