Skip to content

meta: add SENTRY_DEPRECATED(msg) macro#1308

Merged
jpnurmi merged 9 commits intomasterfrom
meta/deprecated
Jul 10, 2025
Merged

meta: add SENTRY_DEPRECATED(msg) macro#1308
jpnurmi merged 9 commits intomasterfrom
meta/deprecated

Conversation

@jpnurmi
Copy link
Copy Markdown
Collaborator

@jpnurmi jpnurmi commented Jul 10, 2025 •
edited
Loading

image 
[build] sentry-native/examples/example.c: In function ‘main’:
[build] sentry-native/examples/example.c:384:5: warning: ‘sentry_sdk_name’ is deprecated: Use `sentry_options_get_sdk_name` instead [-Wdeprecated-declarations]
[build]   384 |     const char *foo = sentry_sdk_name();
[build]       |     ^~~~~
[build] In file included from sentry-native/examples/example.c:9:
[build] sentry-native/include/sentry.h:2593:37: note: declared here
[build]  2593 | SENTRY_EXPERIMENTAL_API const char *sentry_sdk_name(void);
[build]       |                                     ^~~~~~~~~~~~~~~

Docs:

Close: #1307

@jpnurmi jpnurmi mentioned this pull request Jul 10, 2025
Merged
supervacuus
supervacuus approved these changes Jul 10, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@supervacuus supervacuus left a comment

Choose a reason for hiding this comment

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

None of these compilers could compile crashpad, but I still appreciate the version checks.

@jpnurmi jpnurmi marked this pull request as ready for review July 10, 2025 16:13
@jpnurmi jpnurmi merged commit 9ed9a70 into master Jul 10, 2025
58 of 61 checks passed
@jpnurmi jpnurmi deleted the meta/deprecated branch July 10, 2025 16:15
@jpnurmi jpnurmi mentioned this pull request Jul 11, 2025
Merged
@BrewTestBot BrewTestBot mentioned this pull request Aug 8, 2025
Merged
1 task
BernhardMarconato pushed a commit to elgatosf/sentry-native that referenced this pull request Apr 21, 2026
@jpnurmi
meta: add SENTRY_DEPRECATED(msg) macro (getsentry#1308)
4b40b7a 
* meta: add SENTRY_DEPRECATED(msg) macro

* fix deprecation warnings in tests

* Update CHANGELOG.md

* add version checks to be safe

* try to make msvc happy

* add SENTRY_TEST_DEPRECATED

* Revert "fix deprecation warnings in tests"

This reverts commit 0245434.

* Use SENTRY_TEST_DEPRECATED

* fix SENTRY_TEST_DEPRECATED formatting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@supervacuus supervacuus supervacuus approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deprecation macro

2 participants

@jpnurmi @supervacuus