zed icon indicating copy to clipboard operation
zed copied to clipboard

chore: Fix warning from 1.77 rustc

Open osiewicz opened this issue 4 months ago • 1 comments

/cc @maxbrunsfeld , I didn't remove the field outright since I'm not sure if the intent is to use it eventually in extensions work. This is the warning we're getting on 1.77 (release date: 03.21.2024) :

warning: field `0` is never read
  --> crates/language/src/language_registry.rs:81:12
   |
81 |     Loaded(PathBuf, tree_sitter::Language),
   |     ------ ^^^^^^^
   |     |
   |     field in this variant
   |
   = note: `#[warn(dead_code)]` on by default
help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
   |
81 |     Loaded((), tree_sitter::Language),
   |            ~~

warning: field `0` is never read
  --> crates/language/src/language_registry.rs:82:13
   |
82 |     Loading(PathBuf, Vec<oneshot::Sender<Result<tree_sitter::Language>>>),
   |     ------- ^^^^^^^
   |     |
   |     field in this variant
   |
help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
   |
82 |     Loading((), Vec<oneshot::Sender<Result<tree_sitter::Language>>>),
   |             ~~

Release Notes:

  • N/A

osiewicz avatar Feb 23 '24 10:02 osiewicz

@osiewicz Can we just suppress the warning for this line right now?

maxdeviant avatar Feb 23 '24 18:02 maxdeviant