Skip to content

Commit 05f5383

Browse files
Fix warnings in test builds from previous pr
1 parent 3831c2a commit 05f5383

1 file changed

Lines changed: 14 additions & 4 deletions

File tree

editor/src/messages/portfolio/document/overlays/overlays_message_handler.rs

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,12 @@ pub struct OverlaysMessageHandler {
2020
#[message_handler_data]
2121
impl MessageHandler<OverlaysMessage, OverlaysMessageContext<'_>> for OverlaysMessageHandler {
2222
fn process_message(&mut self, message: OverlaysMessage, responses: &mut VecDeque<Message>, context: OverlaysMessageContext) {
23-
let OverlaysMessageContext { visibility_settings, ipp, .. } = context;
24-
let device_pixel_ratio = context.device_pixel_ratio;
23+
let OverlaysMessageContext {
24+
visibility_settings,
25+
ipp,
26+
device_pixel_ratio,
27+
..
28+
} = context;
2529

2630
match message {
2731
#[cfg(target_family = "wasm")]
@@ -68,11 +72,10 @@ impl MessageHandler<OverlaysMessage, OverlaysMessageContext<'_>> for OverlaysMes
6872
}
6973
}
7074
}
71-
#[cfg(test)]
72-
OverlaysMessage::Draw => {}
7375
#[cfg(all(not(target_family = "wasm"), not(test)))]
7476
OverlaysMessage::Draw => {
7577
use super::utility_types::OverlayContext;
78+
7679
let size = ipp.viewport_bounds.size();
7780

7881
let overlay_context = OverlayContext::new(size, device_pixel_ratio, visibility_settings);
@@ -86,6 +89,13 @@ impl MessageHandler<OverlaysMessage, OverlaysMessageContext<'_>> for OverlaysMes
8689
}
8790
responses.add(FrontendMessage::RenderOverlays(overlay_context));
8891
}
92+
#[cfg(all(not(target_family = "wasm"), test))]
93+
OverlaysMessage::Draw => {
94+
// Removes unused warnings in test builds
95+
drop(responses);
96+
drop(context);
97+
drop(super::utility_types::OverlayContext::new(ipp.viewport_bounds.size(), device_pixel_ratio, visibility_settings));
98+
}
8999
OverlaysMessage::AddProvider(message) => {
90100
self.overlay_providers.insert(message);
91101
}

0 commit comments

Comments
 (0)