Skip to content

Commit

Permalink
ref: Remove forward declarations for Envelope (#3732)
Browse files Browse the repository at this point in the history
Having unused forward declarations in the SentryEnvelope.h doesn't make
sense and makes it harder to expose that specific class to Swift.
Therefore, we can remove them.
  • Loading branch information
philipphofmann authored Mar 13, 2024
1 parent 7fe37ab commit d0deebd
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
8 changes: 6 additions & 2 deletions Sources/Sentry/include/HybridPublic/SentryEnvelope.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,12 @@

#endif

@class SentryEvent, SentrySession, SentryId, SentryUserFeedback, SentryAttachment,
SentryTransaction, SentryClientReport, SentryEnvelopeItemHeader;
@class SentryEvent;
@class SentrySession;
@class SentryId;
@class SentryUserFeedback;
@class SentryAttachment;
@class SentryEnvelopeItemHeader;

NS_ASSUME_NONNULL_BEGIN

Expand Down
2 changes: 2 additions & 0 deletions Sources/Sentry/include/SentryEnvelope+Private.h
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

NS_ASSUME_NONNULL_BEGIN

@class SentryClientReport;

@interface
SentryEnvelopeItem ()

Expand Down

0 comments on commit d0deebd

Please sign in to comment.