Replace MSVC-only _ASSERTE with CCP_ASSERT - #41
Open
TrueBrain wants to merge 1 commit into
Open
Conversation
15 tasks
Member
|
Good catch, I don't see why |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This repository, and even the file in question, uses
CCP_ASSERTeverywhere except for these few places.Although they strictly seen do slightly different things, I assume
_ASSERTEis just a left-over, and the intention isCCP_ASSERT.AI assistance disclosure
Claude detected the issue; I fixed the issue.
Type of change
Linked issue (optional)
What changed
Testing
This file isn't included locally. So there is nothing to test as such. You just can't compile the header on MacOS / Linux.
Platforms tested
Screenshots / captures
Checklist
Full disclosure: I am employed by Fenris Creations, although I have no involvement with the Carbon project. I work on this in my free time under my own name.