From a866f8742e4ddf5714455519790fed0f961fad66 Mon Sep 17 00:00:00 2001
From: Héctor Ramón Jiménez <hector0193@gmail.com>
Date: Thu, 22 Jul 2021 20:16:53 +0700
Subject: Avoid cloning `placeholder` for `PickList` unnecessarily during
 `draw`

---
 graphics/src/widget/pick_list.rs | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

(limited to 'graphics')

diff --git a/graphics/src/widget/pick_list.rs b/graphics/src/widget/pick_list.rs
index 01283bd5..88a590b5 100644
--- a/graphics/src/widget/pick_list.rs
+++ b/graphics/src/widget/pick_list.rs
@@ -31,7 +31,7 @@ where
         bounds: Rectangle,
         cursor_position: Point,
         selected: Option<String>,
-        placeholder: Option<String>,
+        placeholder: Option<&str>,
         padding: Padding,
         text_size: u16,
         font: Font,
@@ -70,7 +70,9 @@ where
 
         (
             Primitive::Group {
-                primitives: if let Some(label) = selected.or(placeholder) {
+                primitives: if let Some(label) =
+                    selected.or_else(|| placeholder.map(str::to_string))
+                {
                     let label = Primitive::Text {
                         content: label,
                         size: f32::from(text_size),
-- 
cgit