refactor: migrate to Rust 2021 #16

Merged
fortressia merged 1 commits from refactor/rust2021 into main 2021-12-15 22:09:51 +00:00
fortressia commented 2021-12-13 04:34:55 +00:00 (Migrated from gitlab.com)

Description

Among changes made in Rust 2021, these relevant changes were introduced:

  • The panic!() macro now uses format_args!() like println!().
  • The usage of panic!(x) is now deprecated if x is not a string literal.

Closes

N/A

Implementation Insight

Just run cargo fix --edition and change src/components/editor/mod.rs` line 70 to follow how panics are coded in other parts of the codebase.

TO-DOs

All (should) have been completed.

Additional Information

See: https://doc.rust-lang.org/nightly/edition-guide/rust-2021/panic-macro-consistency.html

# Description Among changes made in Rust 2021, these relevant changes were introduced: - The panic!() macro now uses format_args!() like println!(). - The usage of panic!(x) is now deprecated if x is not a string literal. # Closes N/A # Implementation Insight Just run `cargo fix --edition` and change src/components/editor/mod.rs` line 70 to follow how panics are coded in other parts of the codebase. # TO-DOs All (should) have been completed. # Additional Information See: https://doc.rust-lang.org/nightly/edition-guide/rust-2021/panic-macro-consistency.html
fortressia commented 2021-12-13 04:34:55 +00:00 (Migrated from gitlab.com)

assigned to @fortressia

assigned to @fortressia
fortressia commented 2021-12-15 22:09:38 +00:00 (Migrated from gitlab.com)

changed title from refactor: migrate {-panic!() usage -}to Rust 2021 to refactor: migrate to Rust 2021

changed title from **refactor: migrate {-panic!() usage -}to Rust 2021** to **refactor: migrate to Rust 2021**
fortressia commented 2021-12-15 22:09:51 +00:00 (Migrated from gitlab.com)

mentioned in commit f29b48b9b4

mentioned in commit f29b48b9b402c5f2162a700aca2115ef3d509399
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: fortressia/echidna#16
There is no content yet.