clippy

Author: jstuczyn

clients/client-core/src/config/old_config_v1_1_13.rs

@@ -20,18 +20,18 @@ use std::time::Duration;
 
 #[derive(Debug, Clone, Deserialize, PartialEq, Serialize)]
 #[serde(deny_unknown_fields)]
-pub struct OldConfig_v1_1_13<T> {
+pub struct OldConfigV1_1_13<T> {
     pub client: Client<T>,
 
     #[serde(default)]
     logging: Logging,
     #[serde(default)]
-    debug: OldDebugConfig_v_1_1_13,
+    debug: OldDebugConfigV1_1_13,
 }
 
-impl<T: NymConfig> Default for OldConfig_v1_1_13<T> {
+impl<T: NymConfig> Default for OldConfigV1_1_13<T> {
     fn default() -> Self {
-        OldConfig_v1_1_13 {
+        OldConfigV1_1_13 {
             client: Client::<T>::default(),
             logging: Default::default(),
             debug: Default::default(),
@@ -41,7 +41,7 @@ impl<T: NymConfig> Default for OldConfig_v1_1_13<T> {
 
 #[derive(Debug, Clone, Deserialize, PartialEq, Serialize)]
 #[serde(default, deny_unknown_fields)]
-pub struct OldDebugConfig_v_1_1_13 {
+pub struct OldDebugConfigV1_1_13 {
     #[serde(with = "humantime_serde")]
     pub average_packet_delay: Duration,
 
@@ -96,8 +96,8 @@ pub struct OldDebugConfig_v_1_1_13 {
     pub maximum_reply_key_age: Duration,
 }
 
-impl From<OldDebugConfig_v_1_1_13> for DebugConfig {
-    fn from(value: OldDebugConfig_v_1_1_13) -> Self {
+impl From<OldDebugConfigV1_1_13> for DebugConfig {
+    fn from(value: OldDebugConfigV1_1_13) -> Self {
         DebugConfig {
             traffic: Traffic {
                 average_packet_delay: value.average_packet_delay,
@@ -140,9 +140,9 @@ impl From<OldDebugConfig_v_1_1_13> for DebugConfig {
     }
 }
 
-impl Default for OldDebugConfig_v_1_1_13 {
+impl Default for OldDebugConfigV1_1_13 {
     fn default() -> Self {
-        OldDebugConfig_v_1_1_13 {
+        OldDebugConfigV1_1_13 {
             average_packet_delay: DEFAULT_AVERAGE_PACKET_DELAY,
             average_ack_delay: DEFAULT_AVERAGE_PACKET_DELAY,
             ack_wait_multiplier: DEFAULT_ACK_WAIT_MULTIPLIER,
@@ -169,8 +169,8 @@ impl Default for OldDebugConfig_v_1_1_13 {
     }
 }
 
-impl<T, U> From<OldConfig_v1_1_13<T>> for Config<U> {
-    fn from(value: OldConfig_v1_1_13<T>) -> Self {
+impl<T, U> From<OldConfigV1_1_13<T>> for Config<U> {
+    fn from(value: OldConfigV1_1_13<T>) -> Self {
         Config {
             client: Client {
                 version: value.client.version,

clients/native/src/client/config/old_config_v1_1_13.rs

@@ -2,21 +2,21 @@
 // SPDX-License-Identifier: Apache-2.0
 
 use crate::client::config::{Config, Socket};
-use client_core::config::old_config_v1_1_13::OldConfig_v1_1_13 as OldBaseConfig;
+use client_core::config::old_config_v1_1_13::OldConfigV1_1_13 as OldBaseConfigV1_1_13;
 use nym_config::NymConfig;
 use serde::{Deserialize, Serialize};
 use std::path::PathBuf;
 
 #[derive(Debug, Default, Deserialize, PartialEq, Serialize)]
 #[serde(deny_unknown_fields)]
-pub struct OldConfig_v1_1_13 {
+pub struct OldConfigV1_1_13 {
     #[serde(flatten)]
-    base: OldBaseConfig<OldConfig_v1_1_13>,
+    base: OldBaseConfigV1_1_13<OldConfigV1_1_13>,
 
     socket: Socket,
 }
 
-impl NymConfig for OldConfig_v1_1_13 {
+impl NymConfig for OldConfigV1_1_13 {
     fn template() -> &'static str {
         // not intended to be used
         unimplemented!()
@@ -50,8 +50,8 @@ impl NymConfig for OldConfig_v1_1_13 {
     }
 }
 
-impl From<OldConfig_v1_1_13> for Config {
-    fn from(value: OldConfig_v1_1_13) -> Self {
+impl From<OldConfigV1_1_13> for Config {
+    fn from(value: OldConfigV1_1_13) -> Self {
         Config {
             base: value.base.into(),
             socket: value.socket,

clients/native/src/commands/run.rs

@@ -10,7 +10,7 @@ use crate::{
     error::ClientError,
 };
 
-use crate::client::config::old_config_v1_1_13::OldConfig_v1_1_13;
+use crate::client::config::old_config_v1_1_13::OldConfigV1_1_13;
 use clap::Args;
 use log::*;
 use nym_bin_common::version_checker::is_minor_version_compatible;
@@ -106,7 +106,7 @@ fn load_config(id: &str) -> Result<Config, Box<dyn Error + Send + Sync>> {
             warn!("Failed to load config for {id} - {err}...\nAttempting to use the old config template...");
 
             // if this failed, try to use the old template
-            let old_config = match OldConfig_v1_1_13::load_from_file(id) {
+            let old_config = match OldConfigV1_1_13::load_from_file(id) {
                 Ok(cfg) => {
                     info!("Managed to load config for {id} using old template");
                     cfg

clients/socks5/src/client/config/old_config_v1_1_13.rs

@@ -2,24 +2,24 @@
 // SPDX-License-Identifier: Apache-2.0
 
 use crate::client::config::{Config, Socks5, Socks5Debug};
-use client_core::config::old_config_v1_1_13::OldConfig_v1_1_13 as OldBaseConfig;
+use client_core::config::old_config_v1_1_13::OldConfigV1_1_13 as OldBaseConfigV1_1_13;
 use nym_config::NymConfig;
 use serde::{Deserialize, Serialize};
 use std::path::PathBuf;
 
 #[derive(Clone, Debug, Default, Deserialize, PartialEq, Serialize)]
 #[serde(deny_unknown_fields)]
-pub struct OldConfig_v1_1_13 {
+pub struct OldConfigV1_1_13 {
     #[serde(flatten)]
-    base: OldBaseConfig<OldConfig_v1_1_13>,
+    base: OldBaseConfigV1_1_13<OldConfigV1_1_13>,
 
     socks5: Socks5,
 
     #[serde(default)]
     socks5_debug: Socks5Debug,
 }
 
-impl NymConfig for OldConfig_v1_1_13 {
+impl NymConfig for OldConfigV1_1_13 {
     fn template() -> &'static str {
         // not intended to be used
         unimplemented!()
@@ -55,8 +55,8 @@ impl NymConfig for OldConfig_v1_1_13 {
     }
 }
 
-impl From<OldConfig_v1_1_13> for Config {
-    fn from(value: OldConfig_v1_1_13) -> Self {
+impl From<OldConfigV1_1_13> for Config {
+    fn from(value: OldConfigV1_1_13) -> Self {
         Config {
             base: value.base.into(),
             socks5: value.socks5,

clients/socks5/src/commands/run.rs

@@ -7,7 +7,7 @@ use crate::{
     error::Socks5ClientError,
 };
 
-use crate::client::config::old_config_v1_1_13::OldConfig_v1_1_13;
+use crate::client::config::old_config_v1_1_13::OldConfigV1_1_13;
 use clap::Args;
 use log::*;
 use nym_bin_common::version_checker::is_minor_version_compatible;
@@ -114,7 +114,7 @@ fn load_config(id: &str) -> Result<Config, Box<dyn std::error::Error + Send + Sy
             warn!("Failed to load config for {id} - {err}...\nAttempting to use the old config template...");
 
             // if this failed, try to use the old template
-            let old_config = match OldConfig_v1_1_13::load_from_file(id) {
+            let old_config = match OldConfigV1_1_13::load_from_file(id) {
                 Ok(cfg) => {
                     info!("Managed to load config for {id} using old template");
                     cfg