Skip to content

Commit

Permalink
[WTF] RELEASE_ASSERT_NOT_REACHED should allow a message
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=259526

Reviewed by Mark Lam.

The asserts-enabled and asserts-not-enabled implementations of this macro don't
agree about whether or not it allows a message--allow one for the sake of
form. Since revision f0224c3 it's used in at least one place with a
message.

* Source/WTF/wtf/Assertions.h:

Canonical link: https://commits.webkit.org/266516@main
  • Loading branch information
jjgriego committed Aug 2, 2023
1 parent 9816d96 commit c4574f3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Source/WTF/wtf/Assertions.h
Original file line number Diff line number Diff line change
Expand Up @@ -686,7 +686,7 @@ constexpr bool assertionFailureDueToUnreachableCode = false;
#define RELEASE_ASSERT(assertion, ...) ASSERT(assertion, __VA_ARGS__)
#define RELEASE_ASSERT_WITH_MESSAGE(assertion, ...) ASSERT_WITH_MESSAGE(assertion, __VA_ARGS__)
#define RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(assertion) ASSERT_WITH_SECURITY_IMPLICATION(assertion)
#define RELEASE_ASSERT_NOT_REACHED() ASSERT_NOT_REACHED()
#define RELEASE_ASSERT_NOT_REACHED(...) ASSERT_NOT_REACHED(__VA_ARGS__)
#define RELEASE_ASSERT_NOT_REACHED_UNDER_CONSTEXPR_CONTEXT() ASSERT_NOT_REACHED_UNDER_CONSTEXPR_CONTEXT()
#define RELEASE_ASSERT_UNDER_CONSTEXPR_CONTEXT(assertion) ASSERT_UNDER_CONSTEXPR_CONTEXT(assertion)

Expand Down

0 comments on commit c4574f3

Please sign in to comment.