Skip to content

Update the crates README, clean up dependencies #325

Update the crates README, clean up dependencies

Update the crates README, clean up dependencies #325

GitHub Actions / clippy succeeded Dec 6, 2024 in 1s

clippy

12 warnings

Details

Results

Message level Amount
Internal compiler error 0
Error 0
Warning 12
Note 0
Help 0

Versions

  • rustc 1.83.0 (90b35a623 2024-11-26)
  • cargo 1.83.0 (5ffbef321 2024-10-29)
  • clippy 0.1.83 (90b35a6 2024-11-26)

Annotations

Check warning on line 102 in crates/web-host/src/host/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

function `json_as_var` is never used

warning: function `json_as_var` is never used
   --> crates/web-host/src/host/mod.rs:102:8
    |
102 | pub fn json_as_var(j: &serde_json::Value) -> Result<Var, JsonParseError> {
    |        ^^^^^^^^^^^

Check warning on line 95 in crates/web-host/src/host/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variants `UnknownType`, `UnknownError`, and `InvalidRepresentation` are never constructed

warning: variants `UnknownType`, `UnknownError`, and `InvalidRepresentation` are never constructed
  --> crates/web-host/src/host/mod.rs:95:5
   |
93 | pub enum JsonParseError {
   |          -------------- variants in this enum
94 |     #[error("Unknown type")]
95 |     UnknownType,
   |     ^^^^^^^^^^^
96 |     #[error("Unknown error")]
97 |     UnknownError,
   |     ^^^^^^^^^^^^
98 |     #[error("Invalid representation")]
99 |     InvalidRepresentation,
   |     ^^^^^^^^^^^^^^^^^^^^^
   |
   = note: `JsonParseError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
   = note: `#[warn(dead_code)]` on by default

Check warning on line 24 in crates/daemon/src/rpc_hosts.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

very complex type used. Consider factoring parts into `type` definitions

warning: very complex type used. Consider factoring parts into `type` definitions
  --> crates/daemon/src/rpc_hosts.rs:24:18
   |
24 | pub struct Hosts(HashMap<HostToken, (SystemTime, HostType, Vec<(Obj, SocketAddr)>)>);
   |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity
   = note: `#[warn(clippy::type_complexity)]` on by default

Check warning on line 69 in crates/daemon/src/connections_fjall.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

called `unwrap` on `path` after checking its variant with `is_none`

warning: called `unwrap` on `path` after checking its variant with `is_none`
  --> crates/daemon/src/connections_fjall.rs:69:20
   |
64 |         let (tmpdir, path) = if path.is_none() {
   |                              ----------------- help: try: `if let Some(<item>) = path`
...
69 |             (None, path.unwrap().to_path_buf())
   |                    ^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap
   = note: `#[warn(clippy::unnecessary_unwrap)]` on by default

Check warning on line 584 in crates/kernel/src/tasks/task.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

very complex type used. Consider factoring parts into `type` definitions

warning: very complex type used. Consider factoring parts into `type` definitions
   --> crates/kernel/src/tasks/task.rs:584:6
    |
584 | ) -> Result<Option<((Bytes, VerbDef), Obj)>, CommandError> {
    |      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity
    = note: `#[warn(clippy::type_complexity)]` on by default

Check warning on line 43 in crates/kernel/src/tasks/sessions.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

empty line after doc comment

warning: empty line after doc comment
  --> crates/kernel/src/tasks/sessions.rs:35:1
   |
35 | / ///
36 | | // TODO: Fix up connected/reconnected/discconnected handling.
37 | | //  Will probably deprecate MOO's concept of 'disconnected' and 'connected' players in the long
38 | | //  run and emulate slack, discord, skype, etc which have a concept of 'presence' (online, offline,
...  |
42 | | //  which is different from MOO's "reconnected" handling, but probably preferable.
43 | |
   | |_
44 |   pub trait Session: Send + Sync {
   |   ------------------------------ the comment documents this trait
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments
   = note: `#[warn(clippy::empty_line_after_doc_comments)]` on by default
   = help: if the empty line is unintentional remove it
help: if the doc comment should not document `Session` comment it out
   |
23 ~ // /// The interface for managing the user I/O connection side of state, exposed by the scheduler to
24 ~ // /// the VM during execution and by the host server to the scheduler.
25 ~ // ///
26 ~ // /// Because the execution path within the server is transactional, with the scheduler committing
27 ~ // /// and rolling back 'world state' on task commit/rollback, the general expectation is that this
28 ~ // /// entity should *also* perform transactionally, buffering output until the task commits, and
29 ~ // /// throwing it out on rollback. This may or may not be practical for large amounts of output.
30 ~ // ///
31 ~ // /// It is up to the implementation to decide how to buffer output. Options could include a
32 ~ // /// memory mapped file, a full database, or a simple in-memory buffer.
33 ~ // ///
34 ~ // /// Implementations would live in the 'server' host (e.g. websocket connections or repl loop)
35 ~ // ///
   |

Check warning on line 52 in crates/db/src/tx/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variant name ends with the enum's name

warning: variant name ends with the enum's name
  --> crates/db/src/tx/mod.rs:52:5
   |
52 |     EncodingError,
   |     ^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names

Check warning on line 50 in crates/db/src/tx/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variant name ends with the enum's name

warning: variant name ends with the enum's name
  --> crates/db/src/tx/mod.rs:50:5
   |
50 |     StoreError,
   |     ^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names

Check warning on line 48 in crates/db/src/tx/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variant name ends with the enum's name

warning: variant name ends with the enum's name
  --> crates/db/src/tx/mod.rs:48:5
   |
48 |     RetrieveError,
   |     ^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names
   = note: `#[warn(clippy::enum_variant_names)]` on by default

Check warning on line 28 in crates/db/src/tx/mod.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

non-canonical implementation of `partial_cmp` on an `Ord` type

warning: non-canonical implementation of `partial_cmp` on an `Ord` type
  --> crates/db/src/tx/mod.rs:24:1
   |
24 | /  impl PartialOrd for Timestamp {
25 | |      fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
   | | _______________________________________________________________________-
26 | ||         self.0.partial_cmp(&other.0)
27 | ||     }
   | ||_____- help: change this to: `{ Some(self.cmp(other)) }`
28 | |  }
   | |__^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl
   = note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default

Check warning on line 75 in crates/db/src/worldstate_transaction.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

empty line after doc comment

warning: empty line after doc comment
  --> crates/db/src/worldstate_transaction.rs:74:5
   |
74 | /     /// Get the parent of the given object.
75 | |
   | |_
76 |       fn get_object_parent(&self, obj: &Obj) -> Result<Obj, WorldStateError>;
   |       ----------------------------------------------------------------------- the comment documents this method
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments
   = help: if the empty line is unintentional remove it

Check warning on line 32 in crates/db/src/loader.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

empty line after doc comment

warning: empty line after doc comment
  --> crates/db/src/loader.rs:31:1
   |
31 | / /// WorldState could provide, but potentially different constraints/semantics (e.g. no perms checks)
32 | |
   | |_
33 |   pub trait LoaderInterface: Send {
   |   ------------------------------- the comment documents this trait
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments
   = note: `#[warn(clippy::empty_line_after_doc_comments)]` on by default
   = help: if the empty line is unintentional remove it