firecracker-microvm/firecracker

Use `clippy::as_conversions`

Open

#3,161 opened on 2022年10月4日

GitHub で見る
 (10 comments) (0 reactions) (0 assignees)Rust (34,348 stars) (2,393 forks)batch import
Good first issuePriority: LowStatus: ParkedType: Fix

説明

From a discussion https://github.com/firecracker-microvm/firecracker/pull/3156#discussion_r986640407 it was thought utilizing the clippy lint clippy::as_conversions would introduce additional safety.

It would be good to implement this lint, either via #![warn(clippy::as_conversions)] or passed as a command line argument under test_clippy.py.

To break up this work, we can implement the following lints incrementally:

  • #3195
  • #3196
  • #3197
  • #3198
  • #3199
  • #3200
  • #3201
  • #3202
  • #3203
  • In Cargo.toml replace 
    ptr_as_ptr = "warn"
cast_lossless = "warn"
cast_possible_truncation = "warn"
cast_possible_wrap = "warn"
cast_sign_loss = "warn"
    with 
    as_conversions = "warn"
    and fix warnings.

コントリビューターガイド

技術スタック
rust
領域
toolingdeveloper experience
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Rust programming languageClippy lint tool
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75
調査方針
The remaining work is to modify the `Cargo.toml` file to replace the list of individual clippy lints with `as conversions = "warn"`, and then fix any warnings that arise from enabling this lint. Start by examining the current lint configuration in `Cargo.toml` and the linked discussion at https://github.com/firecracker microvm/firecracker/pull/3156#discussion r986640407 for context. Run `cargo clippy` locally after making the change to identify and resolve new warnings.