Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Setup support for compiler analysis #17

Closed
Tracked by #10
doodspav opened this issue Jun 14, 2023 · 1 comment
Closed
Tracked by #10

Setup support for compiler analysis #17

doodspav opened this issue Jun 14, 2023 · 1 comment
Assignees
@doodspav
Labels
user story User story

Comments

@doodspav
Copy link
Owner

doodspav commented Jun 14, 2023 •
edited
Loading

Description

  • Enable -fsanitize with as many options supported as possible by the relevant compiler
  • Set all compiler warning flags possible (except where contradictory)
  • Make all compiler warnings be errors

Important:

  • Where compiler/linker flags must be set, this shall NOT be done in the CMakeLists.txt file, but shall be done in CMakePresets.json instead

Acceptance Criteria

  • clang-tidy config file exists in root of project
  • -fsanitize is supported by at least one CMake preset per compiler
  • there shall be at least one CMake preset per compiler that treats all warnings as errors in both C90 and the latest version of the C standard supported
@doodspav doodspav mentioned this issue Jun 14, 2023
Open
10 tasks
@doodspav doodspav self-assigned this Jun 14, 2023
@doodspav doodspav added the user story User story label Jun 14, 2023
@doodspav doodspav changed the title Setup support for static analysis and sanitisers Setup support for static analysis and sanitisers with CMake Presets Jun 14, 2023
@doodspav doodspav changed the title Setup support for static analysis and sanitisers with CMake Presets Setup support for compiler analysis Jun 14, 2023
@doodspav
Copy link
Owner Author

Work moved into #20 which has better plan and scope

@doodspav doodspav closed this as not planned Won't fix, can't repro, duplicate, stale Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@doodspav doodspav

Labels
user story User story
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@doodspav