Skip to content

chore: Appease clippy#41

Merged
rbradford merged 6 commits intorust-vmm:mainfrom
RuoqingHe:appease-clippy
Mar 21, 2025
Merged

chore: Appease clippy#41
rbradford merged 6 commits intorust-vmm:mainfrom
RuoqingHe:appease-clippy

Conversation

@RuoqingHe
Copy link
Member

Summary of the PR

  • clippy: Fix needless_lifetimes clippy warning
  • clippy: Fix precedence clippy warning
  • clippy: Fix needless_as_bytes clippy warning
  • clippy: Fix empty_line_after_doc_comments clippy warning
  • clippy: Fix repr_packed_without_abi clippy warning
  • config: Rename config to config.toml

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR have Signed-Off-By trailers (with
    git commit -s), and the commit message has max 60 characters for the
    summary and max 75 characters for each description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

Reported by rust version 1.85.0 (4d91de4e4 2025-02-17). Fix accordingly.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
Reported by rust version 1.85.0 (4d91de4e4 2025-02-17). Fix accordingly.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
Reported by rust version 1.85.0 (4d91de4e4 2025-02-17). Fix accordingly.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
Reported by rust version 1.85.0 (4d91de4e4 2025-02-17). Fix accordingly.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
Reported by rust version 1.85.0 (4d91de4e4 2025-02-17). Fix accordingly.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
As suggested by cargo: `/root/riscv/acpi_tables/.cargo/config` is
deprecated in favor of `config.toml`, rename `config` to `config.toml`.

Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
@rbradford rbradford merged commit 56c71b1 into rust-vmm:main Mar 21, 2025
14 checks passed
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.

3 participants