Remove redundant class string is_empty checks (#844)

This commit is contained in:
Konstantin Itskov 2020-01-05 20:48:51 -05:00 committed by Justin Starry
parent f32aa7e498
commit fb17517e40
2 changed files with 3 additions and 8 deletions

View File

@ -57,6 +57,7 @@ impl Classes {
///
/// Prevents duplication of class names.
pub fn push(&mut self, class: &str) {
let class = class.trim();
if !class.is_empty() {
self.set.insert(class.into());
}

View File

@ -116,10 +116,7 @@ impl VTag {
/// [Element.classList.add](https://developer.mozilla.org/en-US/docs/Web/API/Element/classList)
/// call later.
pub fn add_class(&mut self, class: &str) {
let class = class.trim();
if !class.is_empty() {
self.classes.push(class);
}
self.classes.push(class);
}
/// Adds multiple classes to this virtual node. Actually it will set by
@ -127,10 +124,7 @@ impl VTag {
/// call later.
pub fn add_classes(&mut self, classes: Vec<&str>) {
for class in classes {
let class = class.trim();
if !class.is_empty() {
self.classes.push(class);
}
self.classes.push(class);
}
}