Skip to content

Commit

Permalink
Fix: clippy lint 'uninlined_format_args'
Browse files Browse the repository at this point in the history
  • Loading branch information
13r0ck committed Jan 27, 2023
1 parent e6092e8 commit 2872b7c
Show file tree
Hide file tree
Showing 25 changed files with 46 additions and 51 deletions.
2 changes: 1 addition & 1 deletion examples/download_progress/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ impl Download {
.into()
}
State::Downloading { .. } => {
text(format!("Downloading... {:.2}%", current_progress)).into()
text(format!("Downloading... {current_progress:.2}%")).into()
}
State::Errored => column![
"Something went wrong :(",
Expand Down
2 changes: 1 addition & 1 deletion examples/events/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ impl Application for Events {
let events = Column::with_children(
self.last
.iter()
.map(|event| text(format!("{:?}", event)).size(40))
.map(|event| text(format!("{event:?}")).size(40))
.map(Element::from)
.collect(),
);
Expand Down
2 changes: 1 addition & 1 deletion examples/game_of_life/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ fn view_controls<'a>(

let speed_controls = row![
slider(1.0..=1000.0, speed as f32, Message::SpeedChanged),
text(format!("x{}", speed)).size(16),
text(format!("x{speed}")).size(16),
]
.width(Length::Fill)
.align_items(Alignment::Center)
Expand Down
2 changes: 1 addition & 1 deletion examples/integration_opengl/src/controls.rs
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ impl Program for Controls {
)
.push(sliders)
.push(
Text::new(format!("{:?}", background_color))
Text::new(format!("{background_color:?}"))
.size(14)
.style(Color::WHITE),
),
Expand Down
2 changes: 1 addition & 1 deletion examples/integration_opengl/src/scene.rs
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ impl Scene {
.expect("Cannot create shader");
gl.shader_source(
shader,
&format!("{}\n{}", shader_version, shader_source),
&format!("{shader_version}\n{shader_source}"),
);
gl.compile_shader(shader);
if !gl.get_shader_compile_status(shader) {
Expand Down
2 changes: 1 addition & 1 deletion examples/integration_wgpu/src/controls.rs
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ impl Program for Controls {
)
.push(sliders)
.push(
Text::new(format!("{:?}", background_color))
Text::new(format!("{background_color:?}"))
.size(14)
.style(Color::WHITE),
)
Expand Down
2 changes: 1 addition & 1 deletion examples/integration_wgpu/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ pub fn main() {
}
Err(error) => match error {
wgpu::SurfaceError::OutOfMemory => {
panic!("Swapchain error: {}. Rendering cannot continue.", error)
panic!("Swapchain error: {error}. Rendering cannot continue.")
}
_ => {
// Try rendering again next frame.
Expand Down
7 changes: 3 additions & 4 deletions examples/pokedex/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ impl Application for Pokedex {
Pokedex::Errored { .. } => "Whoops!",
};

format!("{} - Pokédex", subtitle)
format!("{subtitle} - Pokédex")
}

fn update(&mut self, message: Message) -> Command<Message> {
Expand Down Expand Up @@ -158,7 +158,7 @@ impl Pokemon {

let fetch_entry = async {
let url =
format!("https://pokeapi.co/api/v2/pokemon-species/{}", id);
format!("https://pokeapi.co/api/v2/pokemon-species/{id}");

reqwest::get(&url).await?.json().await
};
Expand Down Expand Up @@ -187,8 +187,7 @@ impl Pokemon {

async fn fetch_image(id: u16) -> Result<image::Handle, reqwest::Error> {
let url = format!(
"https://raw.githubusercontent.com/PokeAPI/sprites/master/sprites/pokemon/{}.png",
id
"https://raw.githubusercontent.com/PokeAPI/sprites/master/sprites/pokemon/{id}.png"
);

#[cfg(not(target_arch = "wasm32"))]
Expand Down
2 changes: 1 addition & 1 deletion examples/styling/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ impl Sandbox for Styling {
column![text("Choose a theme:")].spacing(10),
|column, theme| {
column.push(radio(
format!("{:?}", theme),
format!("{theme:?}"),
*theme,
Some(match self.theme {
Theme::Light => ThemeType::Light,
Expand Down
4 changes: 2 additions & 2 deletions examples/system_information/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -114,13 +114,13 @@ impl Application for Example {
{
let memory_readable = ByteSize::kb(memory_used).to_string();

format!("{} kb ({})", memory_used, memory_readable)
format!("{memory_used} kb ({memory_readable})")
} else {
String::from("None")
};

let memory_used =
text(format!("Memory (used): {}", memory_text));
text(format!("Memory (used): {memory_text}"));

let graphics_adapter = text(format!(
"Graphics adapter: {}",
Expand Down
2 changes: 1 addition & 1 deletion examples/todos/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,7 @@ pub enum TaskMessage {

impl Task {
fn text_input_id(i: usize) -> text_input::Id {
text_input::Id::new(format!("task-{}", i))
text_input::Id::new(format!("task-{i}"))
}

fn new(description: String) -> Self {
Expand Down
8 changes: 4 additions & 4 deletions examples/tour/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -388,7 +388,7 @@ impl<'a> Step {

let spacing_section = column![
slider(0..=80, spacing, StepMessage::SpacingChanged),
text(format!("{} px", spacing))
text(format!("{spacing} px"))
.width(Length::Fill)
.horizontal_alignment(alignment::Horizontal::Center),
]
Expand All @@ -412,7 +412,7 @@ impl<'a> Step {
fn text(size: u16, color: Color) -> Column<'a, StepMessage> {
let size_section = column![
"You can change its size:",
text(format!("This text is {} pixels", size)).size(size),
text(format!("This text is {size} pixels")).size(size),
slider(10..=70, size, StepMessage::TextSizeChanged),
]
.padding(20)
Expand All @@ -427,7 +427,7 @@ impl<'a> Step {

let color_section = column![
"And its color:",
text(format!("{:?}", color)).style(color),
text(format!("{color:?}")).style(color),
color_sliders,
]
.padding(20)
Expand Down Expand Up @@ -497,7 +497,7 @@ impl<'a> Step {
.push(ferris(width))
.push(slider(100..=500, width, StepMessage::ImageWidthChanged))
.push(
text(format!("Width: {} px", width))
text(format!("Width: {width} px"))
.width(Length::Fill)
.horizontal_alignment(alignment::Horizontal::Center),
)
Expand Down
2 changes: 1 addition & 1 deletion examples/websocket/src/echo.rs
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ impl fmt::Display for Message {
Message::Disconnected => {
write!(f, "Connection lost... Retrying...")
}
Message::User(message) => write!(f, "{}", message),
Message::User(message) => write!(f, "{message}"),
}
}
}
2 changes: 1 addition & 1 deletion examples/websocket/src/echo/server.rs
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ async fn user_connected(ws: WebSocket) {
tokio::task::spawn(async move {
while let Some(message) = rx.next().await {
user_ws_tx.send(message).await.unwrap_or_else(|e| {
eprintln!("websocket send error: {}", e);
eprintln!("websocket send error: {e}");
});
}
});
Expand Down
2 changes: 1 addition & 1 deletion glow/src/program.rs
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ impl Version {
String::from("#version 120\n#define in varying"),
),
// OpenGL 1.1+
_ => panic!("Incompatible context version: {:?}", version),
_ => panic!("Incompatible context version: {version:?}"),
};

log::info!("Shader directive: {}", vertex.lines().next().unwrap());
Expand Down
6 changes: 3 additions & 3 deletions native/src/command/action.rs
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,10 @@ impl<T> fmt::Debug for Action<T> {
match self {
Self::Future(_) => write!(f, "Action::Future"),
Self::Clipboard(action) => {
write!(f, "Action::Clipboard({:?})", action)
write!(f, "Action::Clipboard({action:?})")
}
Self::Window(action) => write!(f, "Action::Window({:?})", action),
Self::System(action) => write!(f, "Action::System({:?})", action),
Self::Window(action) => write!(f, "Action::Window({action:?})"),
Self::System(action) => write!(f, "Action::System({action:?})"),
Self::Widget(_action) => write!(f, "Action::Widget"),
}
}
Expand Down
8 changes: 4 additions & 4 deletions native/src/debug/basic.rs
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ impl Debug {
}

pub fn log_message<Message: std::fmt::Debug>(&mut self, message: &Message) {
self.last_messages.push_back(format!("{:?}", message));
self.last_messages.push_back(format!("{message:?}"));

if self.last_messages.len() > 10 {
let _ = self.last_messages.pop_front();
Expand All @@ -150,7 +150,7 @@ impl Debug {
let mut lines = Vec::new();

fn key_value<T: std::fmt::Debug>(key: &str, value: T) -> String {
format!("{} {:?}", key, value)
format!("{key} {value:?}")
}

lines.push(format!(
Expand All @@ -176,9 +176,9 @@ impl Debug {
lines.push(String::from("Last messages:"));
lines.extend(self.last_messages.iter().map(|msg| {
if msg.len() <= 100 {
format!(" {}", msg)
format!(" {msg}")
} else {
format!(" {:.100}...", msg)
format!(" {msg:.100}...")
}
}));

Expand Down
4 changes: 2 additions & 2 deletions native/src/image.rs
Original file line number Diff line number Diff line change
Expand Up @@ -107,10 +107,10 @@ pub enum Data {
impl std::fmt::Debug for Data {
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
match self {
Data::Path(path) => write!(f, "Path({:?})", path),
Data::Path(path) => write!(f, "Path({path:?})"),
Data::Bytes(_) => write!(f, "Bytes(...)"),
Data::Rgba { width, height, .. } => {
write!(f, "Pixels({} * {})", width, height)
write!(f, "Pixels({width} * {height})")
}
}
}
Expand Down
2 changes: 1 addition & 1 deletion native/src/svg.rs
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ pub enum Data {
impl std::fmt::Debug for Data {
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
match self {
Data::Path(path) => write!(f, "Path({:?})", path),
Data::Path(path) => write!(f, "Path({path:?})"),
Data::Bytes(_) => write!(f, "Bytes(...)"),
}
}
Expand Down
2 changes: 1 addition & 1 deletion native/src/widget/operation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ where
fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
match self {
Self::None => write!(f, "Outcome::None"),
Self::Some(output) => write!(f, "Outcome::Some({:?})", output),
Self::Some(output) => write!(f, "Outcome::Some({output:?})"),
Self::Chain(_) => write!(f, "Outcome::Chain(...)"),
}
}
Expand Down
11 changes: 5 additions & 6 deletions native/src/window/action.rs
Original file line number Diff line number Diff line change
Expand Up @@ -106,15 +106,14 @@ impl<T> fmt::Debug for Action<T> {
Self::Drag => write!(f, "Action::Drag"),
Self::Resize { width, height } => write!(
f,
"Action::Resize {{ widget: {}, height: {} }}",
width, height
"Action::Resize {{ widget: {width}, height: {height} }}"
),
Self::Maximize(value) => write!(f, "Action::Maximize({})", value),
Self::Minimize(value) => write!(f, "Action::Minimize({}", value),
Self::Maximize(value) => write!(f, "Action::Maximize({value})"),
Self::Minimize(value) => write!(f, "Action::Minimize({value}"),
Self::Move { x, y } => {
write!(f, "Action::Move {{ x: {}, y: {} }}", x, y)
write!(f, "Action::Move {{ x: {x}, y: {y} }}")
}
Self::SetMode(mode) => write!(f, "Action::SetMode({:?})", mode),
Self::SetMode(mode) => write!(f, "Action::SetMode({mode:?})"),
Self::FetchMode(_) => write!(f, "Action::FetchMode"),
Self::ToggleMaximize => write!(f, "Action::ToggleMaximize"),
Self::ToggleDecorations => write!(f, "Action::ToggleDecorations"),
Expand Down
15 changes: 6 additions & 9 deletions src/window/icon.rs
Original file line number Diff line number Diff line change
Expand Up @@ -133,10 +133,9 @@ impl fmt::Display for Error {
Error::InvalidData { byte_count } => {
write!(
f,
"The provided RGBA data (with length {:?}) isn't divisble by \
"The provided RGBA data (with length {byte_count:?}) isn't divisble by \
4. Therefore, it cannot be safely interpreted as 32bpp RGBA \
pixels.",
byte_count,
pixels."
)
}
Error::DimensionsMismatch {
Expand All @@ -146,20 +145,18 @@ impl fmt::Display for Error {
} => {
write!(
f,
"The number of RGBA pixels ({:?}) does not match the provided \
dimensions ({:?}x{:?}).",
pixel_count, width, height,
"The number of RGBA pixels ({pixel_count:?}) does not match the provided \
dimensions ({width:?}x{height:?})."
)
}
Error::OsError(e) => write!(
f,
"The underlying OS failed to create the window \
icon: {:?}",
e
icon: {e:?}"
),
#[cfg(feature = "image_rs")]
Error::ImageError(e) => {
write!(f, "Unable to create icon from a file: {:?}", e)
write!(f, "Unable to create icon from a file: {e:?}")
}
}
}
Expand Down
2 changes: 1 addition & 1 deletion wgpu/src/settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ fn backend_from_env() -> Option<wgpu::Backends> {
"gl" => wgpu::Backends::GL,
"webgpu" => wgpu::Backends::BROWSER_WEBGPU,
"primary" => wgpu::Backends::PRIMARY,
other => panic!("Unknown backend: {}", other),
other => panic!("Unknown backend: {other}"),
}
})
}
2 changes: 1 addition & 1 deletion winit/src/application.rs
Original file line number Diff line number Diff line change
Expand Up @@ -535,7 +535,7 @@ async fn run_instance<A, E, C>(
Err(error) => match error {
// This is an unrecoverable error.
compositor::SurfaceError::OutOfMemory => {
panic!("{:?}", error);
panic!("{error:?}");
}
_ => {
debug.render_finished();
Expand Down
2 changes: 1 addition & 1 deletion winit/src/application/profiler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ impl Profiler {
.unwrap_or("trace");

let path = out_dir
.join(format!("{}_trace_{}.json", curr_exe_name, time));
.join(format!("{curr_exe_name}_trace_{time}.json"));

layer = layer.file(path);
} else {
Expand Down

0 comments on commit 2872b7c

Please sign in to comment.