From 8b88bf9ba28526dac5be98891b6068d820b8ed88 Mon Sep 17 00:00:00 2001 From: Alexandre Bury Date: Fri, 27 Jan 2017 14:51:46 -0800 Subject: [PATCH] Add `Menubar::is_empty(&self)` And fix another clippy warning --- src/menu.rs | 13 +++++++------ src/views/menubar.rs | 5 +++++ 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/src/menu.rs b/src/menu.rs index 4f1be7f..37425af 100644 --- a/src/menu.rs +++ b/src/menu.rs @@ -75,11 +75,6 @@ impl MenuTree { self.children.clear(); } - /// Returns `true` if this tree has no children. - pub fn is_empty(&self) -> bool { - self.children.is_empty() - } - /// Inserts a delimiter at the given position. pub fn insert_delimiter(&mut self, i: usize) { self.children.insert(i, MenuItem::Delimiter); @@ -171,7 +166,7 @@ impl MenuTree { /// or if it wasn't a subtree. pub fn find_subtree(&mut self, title: &str) -> Option<&mut MenuTree> { self.find_item(title).and_then(|item| { - if let &mut MenuItem::Subtree(_, ref mut tree) = item { + if let MenuItem::Subtree(_, ref mut tree) = *item { Some(Rc::make_mut(tree)) } else { None @@ -191,4 +186,10 @@ impl MenuTree { pub fn len(&self) -> usize { self.children.len() } + + /// Returns `true` if this tree has no children. + pub fn is_empty(&self) -> bool { + self.children.is_empty() + } + } diff --git a/src/views/menubar.rs b/src/views/menubar.rs index 6621602..afe1469 100644 --- a/src/views/menubar.rs +++ b/src/views/menubar.rs @@ -99,6 +99,11 @@ impl Menubar { self.menus.len() } + /// Returns `true` if this menubar is empty. + pub fn is_empty(&self) -> bool { + self.menus.is_empty() + } + /// Returns the item at the given position. /// /// Returns `None` if `i > self.len()`