FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

main.cpp: error out when file/path provided by `-I` or `-include=`, or the input does not exist by firewave · Pull Request #435 · cppcheck-opensource/simplecpp · GitHub

main.cpp: error out when file/path provided by -I or -include=, or the input does not exist - #435

Merged
firewave merged 3 commits into
cppcheck-opensource:masterfrom
firewave:main-err
Oct 1, 2025
Merged

main.cpp: error out when file/path provided by -I or -include=, or the input does not exist#435
firewave merged 3 commits into
cppcheck-opensource:masterfrom
firewave:main-err

Conversation

Copy link
Copy Markdown
Collaborator

No description provided.

firewave marked this pull request as draft June 11, 2025 10:55
firewave changed the title main.cpp: error out when file/path provided by -I or -include= do not exist main.cpp: error out when file/path provided by -I or -include= does not exist Jun 11, 2025
firewave force-pushed the main-err branch 4 times, most recently from 807bbf5 to 97e7735 Compare September 10, 2025 11:57
firewave changed the title main.cpp: error out when file/path provided by -I or -include= does not exist main.cpp: error out when file/path provided by -I or -include=, or the input do not exist Sep 10, 2025
firewave force-pushed the main-err branch 2 times, most recently from e8947d4 to a179a19 Compare September 10, 2025 12:13

Copy link
Copy Markdown
Collaborator Author

It will only fail when -is is specified. The check within simplecpp is still flawed.

These checks are important for #438. The code can be simplified if #339 has been merged.

Copy link
Copy Markdown
Collaborator Author

It will only fail when -is is specified. The check within simplecpp is still flawed.

I added checks for all cases in the CLI. I filed #539 about improving the library.

firewave marked this pull request as ready for review September 10, 2025 12:30
firewave mentioned this pull request Sep 10, 2025
firewave marked this pull request as draft September 10, 2025 12:59
firewave marked this pull request as ready for review September 10, 2025 13:09
firewave changed the title main.cpp: error out when file/path provided by -I or -include=, or the input do not exist main.cpp: error out when file/path provided by -I or -include=, or the input does not exist Sep 10, 2025
firewave merged commit dcc4fdd into cppcheck-opensource:master Oct 1, 2025
16 checks passed
firewave deleted the main-err branch October 1, 2025 15:28
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL