Skip to content
Permalink
Browse files

replace len() !=0 with !is_empty()

  • Loading branch information...
jxs committed Aug 17, 2015
1 parent 3bb9306 commit 1e2044774645424b555bf4ff789c3e686ab67466
Showing with 3 additions and 4 deletions.
  1. +1 −1 components/gfx/display_list/mod.rs
  2. +1 −1 components/layout/flow.rs
  3. +1 −2 tests/unit/net/hsts.rs
@@ -213,7 +213,7 @@ impl DisplayList {
};

doit(&(self.all_display_items()));
if self.children.len() != 0 {
if !self.children.is_empty() {
println!("{} Children stacking contexts list length: {}",
indentation,
self.children.len());
@@ -244,7 +244,7 @@ pub trait Flow: fmt::Debug + Sync {
match self.class() {
FlowClass::Block |
FlowClass::TableCaption |
FlowClass::TableCell if base(self).children.len() != 0 => {
FlowClass::TableCell if !base(self).children.is_empty() => {
// FIXME(#2795): Get the real container size.
let container_size = Size2D::zero();
for kid in mut_base(self).children.iter_mut() {
@@ -275,7 +275,7 @@ fn test_hsts_list_with_expired_entry_is_not_is_host_secure() {
#[test]
fn test_preload_hsts_domains_well_formed() {
let hsts_list = preload_hsts_domains().unwrap();
assert!(hsts_list.entries.len() != 0);
assert!(!hsts_list.entries.is_empty());
}

#[test]
@@ -301,4 +301,3 @@ fn test_secure_url_forces_an_http_host_in_list_to_https() {

assert_eq!(&secure.scheme, "https");
}

0 comments on commit 1e20447

Please sign in to comment.
You can’t perform that action at this time.