Skip to content

Commit

Permalink
Move '&&' to the end of the previous line.
Browse files Browse the repository at this point in the history
Following #10692 this is just a
formating change to satisfy a new tidy requirement of not having '&&' at
the beginning of a line.
  • Loading branch information
zwn committed Apr 22, 2016
1 parent 01b111c commit 924d804
Show file tree
Hide file tree
Showing 11 changed files with 29 additions and 29 deletions.
6 changes: 3 additions & 3 deletions components/gfx/text/glyph.rs
Expand Up @@ -469,9 +469,9 @@ impl<'a> GlyphStore {
character: char,
data: &GlyphData) {
let glyph_is_compressible = is_simple_glyph_id(data.id) &&
is_simple_advance(data.advance)
&& data.offset == Point2D::zero()
&& data.cluster_start; // others are stored in detail buffer
is_simple_advance(data.advance) &&
data.offset == Point2D::zero() &&
data.cluster_start; // others are stored in detail buffer

debug_assert!(data.ligature_start); // can't compress ligature continuation glyphs.
debug_assert!(i < self.char_len());
Expand Down
4 changes: 2 additions & 2 deletions components/gfx/text/shaping/harfbuzz.rs
Expand Up @@ -412,8 +412,8 @@ impl Shaper {

let mut covered_byte_span = char_byte_span.clone();
// extend, clipping at end of text range.
while covered_byte_span.end() < byte_max
&& byte_to_glyph[covered_byte_span.end()] == NO_GLYPH {
while covered_byte_span.end() < byte_max &&
byte_to_glyph[covered_byte_span.end()] == NO_GLYPH {
let range = text.char_range_at(covered_byte_span.end());
drop(range.ch);
covered_byte_span.extend_to(range.next);
Expand Down
4 changes: 2 additions & 2 deletions components/layout/traversal.rs
Expand Up @@ -194,9 +194,9 @@ impl<'a> PostorderFlowTraversal for AssignBSizes<'a> {
#[inline]
fn should_process(&self, flow: &mut Flow) -> bool {
let base = flow::base(flow);
base.restyle_damage.intersects(REFLOW_OUT_OF_FLOW | REFLOW)
base.restyle_damage.intersects(REFLOW_OUT_OF_FLOW | REFLOW) &&
// The fragmentation countainer is responsible for calling Flow::fragment recursively
&& !base.flags.contains(CAN_BE_FRAGMENTED)
!base.flags.contains(CAN_BE_FRAGMENTED)
}
}

Expand Down
8 changes: 4 additions & 4 deletions components/net/cookie.rs
Expand Up @@ -136,10 +136,10 @@ impl Cookie {
if string == domain_string {
return true;
}
if string.ends_with(domain_string)
&& string.as_bytes()[string.len()-domain_string.len()-1] == b'.'
&& string.parse::<Ipv4Addr>().is_err()
&& string.parse::<Ipv6Addr>().is_err() {
if string.ends_with(domain_string) &&
string.as_bytes()[string.len()-domain_string.len()-1] == b'.' &&
string.parse::<Ipv4Addr>().is_err() &&
string.parse::<Ipv6Addr>().is_err() {
return true;
}
false
Expand Down
4 changes: 2 additions & 2 deletions components/plugins/lints/transmute_type.rs
Expand Up @@ -28,8 +28,8 @@ impl LateLintPass for TransmutePass {
match expr.node {
hir::ExprPath(_, ref path) => {
if path.segments.last()
.map_or(false, |ref segment| segment.identifier.name.as_str() == "transmute")
&& args.len() == 1 {
.map_or(false, |ref segment| segment.identifier.name.as_str() == "transmute") &&
args.len() == 1 {
let tcx = cx.tcx;
cx.span_lint(TRANSMUTE_TYPE_LINT, ex.span,
&format!("Transmute to {:?} from {:?} detected",
Expand Down
4 changes: 2 additions & 2 deletions components/script/dom/htmlcollection.rs
Expand Up @@ -161,8 +161,8 @@ impl HTMLCollection {
}
impl CollectionFilter for TagNameNSFilter {
fn filter(&self, elem: &Element, _root: &Node) -> bool {
((self.qname.ns == Namespace(atom!("*"))) || (self.qname.ns == *elem.namespace()))
&& ((self.qname.local == atom!("*")) || (self.qname.local == *elem.local_name()))
((self.qname.ns == Namespace(atom!("*"))) || (self.qname.ns == *elem.namespace())) &&
((self.qname.local == atom!("*")) || (self.qname.local == *elem.local_name()))
}
}
let filter = TagNameNSFilter {
Expand Down
4 changes: 2 additions & 2 deletions components/script/dom/htmlformelement.rs
Expand Up @@ -244,8 +244,8 @@ impl HTMLFormElement {
let base = doc.url();
// TODO: Handle browsing contexts
// Step 4
if submit_method_flag == SubmittedFrom::NotFromFormSubmitMethod
&& !submitter.no_validate(self)
if submit_method_flag == SubmittedFrom::NotFromFormSubmitMethod &&
!submitter.no_validate(self)
{
if self.interactive_validation().is_err() {
// TODO: Implement event handlers on all form control elements
Expand Down
10 changes: 5 additions & 5 deletions components/script/dom/htmltableelement.rs
Expand Up @@ -132,8 +132,8 @@ impl HTMLTableElementMethods for HTMLTableElement {

impl CollectionFilter for TableRowFilter {
fn filter(&self, elem: &Element, root: &Node) -> bool {
elem.is::<HTMLTableRowElement>()
&& (root.is_parent_of(elem.upcast())
elem.is::<HTMLTableRowElement>() &&
(root.is_parent_of(elem.upcast())
|| self.sections.iter().any(|ref section| section.is_parent_of(elem.upcast())))
}
}
Expand Down Expand Up @@ -250,9 +250,9 @@ impl HTMLTableElementMethods for HTMLTableElement {
struct TBodiesFilter;
impl CollectionFilter for TBodiesFilter {
fn filter(&self, elem: &Element, root: &Node) -> bool {
elem.is::<HTMLTableSectionElement>()
&& elem.local_name() == &atom!("tbody")
&& elem.upcast::<Node>().GetParentNode().r() == Some(root)
elem.is::<HTMLTableSectionElement>() &&
elem.local_name() == &atom!("tbody") &&
elem.upcast::<Node>().GetParentNode().r() == Some(root)
}
}

Expand Down
4 changes: 2 additions & 2 deletions components/script/dom/htmltablerowelement.rs
Expand Up @@ -29,8 +29,8 @@ use util::str::DOMString;
struct CellsFilter;
impl CollectionFilter for CellsFilter {
fn filter(&self, elem: &Element, root: &Node) -> bool {
(elem.is::<HTMLTableHeaderCellElement>() || elem.is::<HTMLTableDataCellElement>())
&& elem.upcast::<Node>().GetParentNode().r() == Some(root)
(elem.is::<HTMLTableHeaderCellElement>() || elem.is::<HTMLTableDataCellElement>()) &&
elem.upcast::<Node>().GetParentNode().r() == Some(root)
}
}

Expand Down
4 changes: 2 additions & 2 deletions components/script/dom/htmltablesectionelement.rs
Expand Up @@ -44,8 +44,8 @@ impl HTMLTableSectionElement {
struct RowsFilter;
impl CollectionFilter for RowsFilter {
fn filter(&self, elem: &Element, root: &Node) -> bool {
elem.is::<HTMLTableRowElement>()
&& elem.upcast::<Node>().GetParentNode().r() == Some(root)
elem.is::<HTMLTableRowElement>() &&
elem.upcast::<Node>().GetParentNode().r() == Some(root)
}
}

Expand Down
6 changes: 3 additions & 3 deletions support/android/build-apk/src/main.rs
Expand Up @@ -274,9 +274,9 @@ fn find_native_libs(args: &Args) -> HashMap<String, PathBuf> {
(Some(file_name), Some(extension)) => {
let file_name = file_name.to_str().unwrap();

if file_name.starts_with("lib")
&& extension == "so"
&& args.shared_libraries.contains(file_name) {
if file_name.starts_with("lib") &&
extension == "so" &&
args.shared_libraries.contains(file_name) {
println!("Adding the file {:?}", file_name);
native_shared_libs.insert(file_name.to_string(), path.to_path_buf().clone());
break;
Expand Down

0 comments on commit 924d804

Please sign in to comment.