Rustfmt and clippy fixes for newer version of rustfmt/clippy (#566)

* Update formatting to newer version of rustfmt

Signed-off-by: Nico Burns <nico@nicoburns.com>

* Fix clippy lints

* Ignore clippy::empty_line_after_doc_comments lint

Signed-off-by: Nico Burns <nico@nicoburns.com>

---------

Signed-off-by: Nico Burns <nico@nicoburns.com>

Author: nicoburns

html5ever/examples/arena.rs

@@ -183,7 +183,10 @@ impl<'arena> Sink<'arena> {
 impl<'arena> TreeSink for Sink<'arena> {
     type Handle = Ref<'arena>;
     type Output = Ref<'arena>;
-    type ElemName<'a> = &'a QualName where Self : 'a;
+    type ElemName<'a>
+        = &'a QualName
+    where
+        Self: 'a;
 
     fn finish(self) -> Ref<'arena> {
         self.document

markup5ever/interface/mod.rs

@@ -26,7 +26,7 @@ pub struct ExpandedName<'a> {
     pub local: &'a LocalName,
 }
 
-impl<'a> ElemName for ExpandedName<'a> {
+impl ElemName for ExpandedName<'_> {
     #[inline(always)]
     fn ns(&self) -> &Namespace {
         self.ns
@@ -50,7 +50,7 @@ impl<'a> ElemName for Ref<'a, ExpandedName<'a>> {
     }
 }
 
-impl<'a> fmt::Debug for ExpandedName<'a> {
+impl fmt::Debug for ExpandedName<'_> {
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         if self.ns.is_empty() {
             write!(f, "{}", self.local)
@@ -266,7 +266,7 @@ pub struct QualName {
     pub local: LocalName,
 }
 
-impl<'a> ElemName for Ref<'a, QualName> {
+impl ElemName for Ref<'_, QualName> {
     #[inline(always)]
     fn ns(&self) -> &Namespace {
         &self.ns
@@ -278,7 +278,7 @@ impl<'a> ElemName for Ref<'a, QualName> {
     }
 }
 
-impl<'a> ElemName for &'a QualName {
+impl ElemName for &QualName {
     #[inline(always)]
     fn ns(&self) -> &Namespace {
         &self.ns

markup5ever/lib.rs

@@ -8,6 +8,8 @@
 // except according to those terms.
 
 #![allow(unexpected_cfgs)]
+// This error is coming from code generated by PHF which we cannot directly fix
+#![allow(clippy::empty_line_after_doc_comments)]
 
 pub use tendril;
 

rcdom/lib.rs

@@ -225,7 +225,10 @@ impl TreeSink for RcDom {
 
     type Handle = Handle;
 
-    type ElemName<'a> = ExpandedName<'a> where Self : 'a;
+    type ElemName<'a>
+        = ExpandedName<'a>
+    where
+        Self: 'a;
 
     fn parse_error(&self, msg: Cow<'static, str>) {
         self.errors.borrow_mut().push(msg);
@@ -260,10 +263,10 @@ impl TreeSink for RcDom {
     }
 
     fn elem_name<'a>(&self, target: &'a Handle) -> ExpandedName<'a> {
-        return match target.data {
+        match target.data {
             NodeData::Element { ref name, .. } => name.expanded(),
             _ => panic!("not an element!"),
-        };
+        }
     }
 
     fn create_element(&self, name: QualName, attrs: Vec<Attribute>, flags: ElementFlags) -> Handle {

xml5ever/src/tokenizer/qname.rs

@@ -20,7 +20,7 @@ pub struct QualNameTokenizer<'a> {
     curr_ind: usize,
 }
 
-impl<'a> QualNameTokenizer<'a> {
+impl QualNameTokenizer<'_> {
     pub fn new(tag: &[u8]) -> QualNameTokenizer {
         QualNameTokenizer {
             state: QualNameState::BeforeName,