omicron/oas
1
0
Fork 0
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

various small fixes #20

Merged
omicron merged 5 commits from small_fixes into main 2025-04-17 14:47:14 +00:00
Conversation 1 Commits 5 Files Changed 6 +7 -13
omicron commented 2025-04-17 13:17:55 +00:00
Owner
Copy Link

Various small cleanups and fixing warnings

  • Add new /build directory to .gitignore and remove the old build artifacts that no longer get created
  • Fix a warning about comparison of different sign in lexer_shift_buffer
  • Remove an unused function that had a parameter warning
Various small cleanups and fixing warnings - Add new `/build` directory to .gitignore and remove the old build artifacts that no longer get created - Fix a warning about comparison of different sign in `lexer_shift_buffer` - Remove an unused function that had a parameter warning
omicron added 3 commits 2025-04-17 13:17:55 +00:00
Update gitignore, add /build and remove old build artifacts 26cb374c1d
Remove unused ast_node_free_value 1a79bf050e 
Values are all inside the ast struct and require no cleanup other than
freeing the ast struct.
Change the n argument of lexer_shift_buffer to size_t from int
Some checks failed
Validate the build / validate-build (push) Failing after 35s
Details
6f78d26ea1
omicron added 2 commits 2025-04-17 14:37:52 +00:00
Let the release build error on warnings bf3fd83b64 
Add -Werror to the release configuration. Also add the release build as
a dependency of the make validate rule. The idea is that builds should
not pass validation if they have warnings but it shouldn't stop debug
builds during development from compiling while work is in progress.
Make compiler-rt version match the clang version in the gitea action
Some checks failed
Validate the build / validate-build (push) Failing after 5s
Details
ceabd22b70
omicron force-pushed small_fixes from ceabd22b70 to c0025eb4d3 2025-04-17 14:39:16 +00:00 Compare
omicron force-pushed small_fixes from c0025eb4d3 to ea5164e584 2025-04-17 14:44:01 +00:00 Compare
omicron commented 2025-04-17 14:47:05 +00:00
Author
Owner
Copy Link

Release builds now fail on warnings and the release configuration is added as a dependency to make validate. The idea is that debug builds during development can warn but make validate will fail when there are warnings.

The correct clang version is picked to match the compiler-rt version in alpine edge repositories.

Release builds now fail on warnings and the release configuration is added as a dependency to `make validate`. The idea is that debug builds during development can warn but `make validate` will fail when there are warnings. The correct clang version is picked to match the compiler-rt version in alpine edge repositories.
omicron merged commit ea5164e584 into main 2025-04-17 14:47:14 +00:00
omicron deleted branch small_fixes 2025-04-17 14:47:14 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
component
lexer
critical
feature
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
omicron
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: omicron/oas#20
No description provided.
Delete Branch "small_fixes"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?