clean code

adbar · Oct 16, 2023 · c39b9d2 · c39b9d2
1 parent d070356
commit c39b9d2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/htmldate/settings.py b/htmldate/settings.py
@@ -39,6 +39,7 @@
     "picture",
     "rdf",
     "svg",
+    "track",
     "video",
 ]
-# "iframe", "layer", "param"
+# "iframe", "input", "layer", "param", "source"
diff --git a/htmldate/utils.py b/htmldate/utils.py
@@ -23,7 +23,6 @@
     cchardet_detect = None
 from charset_normalizer import from_bytes
 
-# from lxml.etree import strip_tags
 from lxml.html import HtmlElement, HTMLParser, fromstring  # type: ignore
 
 from .settings import MAX_FILE_SIZE, MIN_FILE_SIZE
@@ -219,19 +218,11 @@ def load_html(htmlobject: Union[bytes, str, HtmlElement]) -> Optional[HtmlElemen
     return tree
 
 
-def clean_html(tree, elemlist):
+def clean_html(tree: HtmlElement, elemlist: List[str]) -> HtmlElement:
     "Delete selected elements."
-    # return tree
     for element in tree.iter(elemlist):
         try:
             element.drop_tree()
-        except AttributeError:
+        except AttributeError:  # pragma: no cover
             element.getparent().remove(element)
-    # for element in tree.xpath(".//*[@src]"):
-    #    try:
-    #        element.drop_tree()
-    #    except AttributeError:
-    #        element.getparent().remove(element)
-    # for element in tree.iter("head", "html", "title"):
-    #    element.drop_tag()
     return tree