Skip to content

Conversation

firewave
Copy link
Collaborator

No description provided.

@firewave firewave marked this pull request as ready for review January 19, 2023 00:10
@firewave
Copy link
Collaborator Author

The changed test result is expected since when I wrote the test I assumed that "system header may be relative to source file" is actually valid. Since this has been (correctly) removed from simplecpp the test failed.

That fix has been extracted from #4704 which will increase the test coverage of missing includes.

@danmar danmar merged commit 99acd31 into danmar:main Jan 20, 2023
@firewave firewave deleted the simplecpp branch January 20, 2023 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants