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

Fix unsafe undefined behavior and update deps #153

Merged
merged 5 commits into from
Dec 26, 2024
Merged

Fix unsafe undefined behavior and update deps #153

merged 5 commits into from
Dec 26, 2024

Conversation

alloncm
Copy link
Owner

@alloncm alloncm commented Dec 25, 2024

  • Fixed an undefined behavior found with Miri in fixed_size_queue
  • Remove unused development dependencies by using only the features I need (this decreased the amount of crates needed and boosted compilation time)
  • Update dependencies (reqwest, chrono) to address vulnerability alerts
  • Delete stack_string with arrayvec's ArrayString
  • Add rust-analyzer to the rust-toolchain.toml file
    rust-analyzer is promised
    to work only on the latest rust release
    and since I have pinned an older version I need to
    also download an older rust-analyzer version

@alloncm
Fix UB in fixed_size_queue and move tests to the module itself
db5fd54
@alloncm
Fix tests compilation
bd50e8b
@alloncm
Remove unused dev deps
9746845
@alloncm
Replace stack_string with arrayvec
0c95930 
Also update vulnerable
deps and fix compilation error
@alloncm
Add rust-analyzer to rust-toolchain.toml
70cde66 
Add this component since rust-analyzer is promised
to work only on the latest rust release
and since I have pinned an older version I need to
also download an older rust-analyzer version
@alloncm alloncm changed the title Fix unsafe undefined behavior Fix unsafe undefined behavior and update deps Dec 25, 2024
@alloncm alloncm merged commit 48c7340 into master Dec 26, 2024
2 checks passed
@alloncm alloncm deleted the fix/unsafe_ub branch December 26, 2024 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@alloncm