Autoformat and bump version

Author: jponttuset

README.md

@@ -120,7 +120,7 @@ patterns.
 ## Usage:
 
 ```
-usage: [email protected].6  [-h] [--resize_images] [--im_size IM_SIZE]
+usage: [email protected].7  [-h] [--resize_images] [--im_size IM_SIZE]
                                    [--compress_pdf]
                                    [--pdf_im_resolution PDF_IM_RESOLUTION]
                                    [--images_allowlist IMAGES_ALLOWLIST]

arxiv_latex_cleaner/_version.py

@@ -13,4 +13,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-__version__ = "v1.0.6"
+__version__ = "v1.0.7"

arxiv_latex_cleaner/arxiv_latex_cleaner.py

@@ -187,82 +187,82 @@ def _simplify_conditional_blocks(text):
   Raises a ValueError if an unmatched \if or \else is found.
   """
   p = regex.compile(r'(?!(?<=\\newif\s*))\\if\s*(\w+)|\\else(?!\w)|\\fi(?!\w)')
-  toplevel_tree = { "left": [], "kind": "toplevel", "parent": None }
+  toplevel_tree = {'left': [], 'kind': 'toplevel', 'parent': None}
 
   tree = toplevel_tree
 
   def new_subtree(kind):
-    return {  "kind": kind, "left": [], "right": [] }
-  
+    return {'kind': kind, 'left': [], 'right': []}
+
   def add_subtree(tree, subtree):
-    if "else" not in tree:
-      tree["left"].append(subtree)
+    if 'else' not in tree:
+      tree['left'].append(subtree)
     else:
-      tree["right"].append(subtree)
-    subtree["parent"] = tree
+      tree['right'].append(subtree)
+    subtree['parent'] = tree
 
   for m in p.finditer(text):
     m_no_space = m.group().replace(' ', '')
-    if (m_no_space == r'\iffalse' or m_no_space == r'\if0'):
-      subtree = new_subtree("iffalse")
-      subtree["start"] = m
+    if m_no_space == r'\iffalse' or m_no_space == r'\if0':
+      subtree = new_subtree('iffalse')
+      subtree['start'] = m
       add_subtree(tree, subtree)
       tree = subtree
-    elif (m_no_space == r'\iftrue' or m_no_space == r'\if1'):
-      subtree = new_subtree("iftrue")
-      subtree["start"] = m
+    elif m_no_space == r'\iftrue' or m_no_space == r'\if1':
+      subtree = new_subtree('iftrue')
+      subtree['start'] = m
       add_subtree(tree, subtree)
       tree = subtree
     elif m_no_space.startswith(r'\if'):
-      subtree = new_subtree("unknown")
-      subtree["start"] = m
+      subtree = new_subtree('unknown')
+      subtree['start'] = m
       add_subtree(tree, subtree)
       tree = subtree
-    elif (m_no_space == r'\else'):
-      if tree["parent"] is None:
+    elif m_no_space == r'\else':
+      if tree['parent'] is None:
         os.sys.stderr.write('Warning: Ignoring unmatched \\else!\n')
         continue
       tree['else'] = m
     elif m.group() == r'\fi':
-      if tree["parent"] is None:
+      if tree['parent'] is None:
         os.sys.stderr.write('Warning: Ignoring unmatched \\fi!\n')
         continue
-      tree["end"] = m
-      tree = tree["parent"]
+      tree['end'] = m
+      tree = tree['parent']
     else:
       raise ValueError('Unreachable!')
-  if tree["parent"] is not None:
+  if tree['parent'] is not None:
     raise ValueError(f"Unmatched {tree['start'].group()}")
 
   positions_to_delete = []
-  
+
   def traverse_tree(tree):
-    if tree["kind"] == "iffalse":
-      if "else" in tree:
-        positions_to_delete.append((tree["start"].start(), tree["else"].end()))
-        for subtree in tree["right"]:
+    if tree['kind'] == 'iffalse':
+      if 'else' in tree:
+        positions_to_delete.append((tree['start'].start(), tree['else'].end()))
+        for subtree in tree['right']:
           traverse_tree(subtree)
-        positions_to_delete.append((tree["end"].start(), tree["end"].end()))
+        positions_to_delete.append((tree['end'].start(), tree['end'].end()))
       else:
-        positions_to_delete.append((tree["start"].start(), tree["end"].end()))
-    elif tree["kind"] == "iftrue":
-      if "else" in tree:
-        positions_to_delete.append((tree["start"].start(), tree["start"].end()))
-        for subtree in tree["left"]:
+        positions_to_delete.append((tree['start'].start(), tree['end'].end()))
+    elif tree['kind'] == 'iftrue':
+      if 'else' in tree:
+        positions_to_delete.append((tree['start'].start(), tree['start'].end()))
+        for subtree in tree['left']:
           traverse_tree(subtree)
-        positions_to_delete.append((tree["else"].start(), tree["end"].end()))
+        positions_to_delete.append((tree['else'].start(), tree['end'].end()))
       else:
-        positions_to_delete.append((tree["start"].start(), tree["start"].end()))
-        positions_to_delete.append((tree["end"].start(), tree["end"].end()))
-    elif tree["kind"] == "unknown":
-      for subtree in tree["left"]:
+        positions_to_delete.append((tree['start'].start(), tree['start'].end()))
+        positions_to_delete.append((tree['end'].start(), tree['end'].end()))
+    elif tree['kind'] == 'unknown':
+      for subtree in tree['left']:
         traverse_tree(subtree)
-      for subtree in tree["right"]:
+      for subtree in tree['right']:
         traverse_tree(subtree)
     else:
       raise ValueError('Unreachable!')
 
-  for tree in toplevel_tree["left"]:
+  for tree in toplevel_tree['left']:
     traverse_tree(tree)
 
   for start, end in reversed(positions_to_delete):

arxiv_latex_cleaner/tests/arxiv_latex_cleaner_test.py

@@ -473,8 +473,8 @@ def test_remove_environment(self, text_in, true_output):
       },
       {
           'testcase_name': 'commands_not_removed',
-          'text_in': '\\newcommand\\figref[1]{Figure~\\ref{fig:\#1}}',
-          'true_output': '\\newcommand\\figref[1]{Figure~\\ref{fig:\#1}}',
+          'text_in': '\\newcommand\\figref[1]{Figure~\\ref{fig:\\#1}}',
+          'true_output': '\\newcommand\\figref[1]{Figure~\\ref{fig:\\#1}}',
       },
       {
           'testcase_name': 'iffalse_else_sustained',
@@ -498,7 +498,7 @@ def test_remove_environment(self, text_in, true_output):
       },
       {
           'testcase_name': 'new_if_ignored',
-          'text_in': '\\newif  \\ifvar \\ifvar\iffalse test\\fi\\fi',
+          'text_in': '\\newif  \\ifvar \\ifvar\\iffalse test\\fi\\fi',
           'true_output': '\\newif  \\ifvar \\ifvar\\fi',
       },
   )
@@ -649,9 +649,11 @@ def test_replace_tikzpictures(self, text_in, figures_in, true_output):
           ],
           'true_output': 'Foo\\includeinkscape[width=\\linewidth]{ext_svg/test2_svg-tex.pdf_tex}\nFoo',
       },
-       {
+      {
           'testcase_name': 'includesvg_match_with_options_with_dot_with_suffix',
-          'text_in': 'Foo\\includesvg[width=\\linewidth]{figs/test2-0.9.svg}\nFoo',
+          'text_in': (
+              'Foo\\includesvg[width=\\linewidth]{figs/test2-0.9.svg}\nFoo'
+          ),
           'figures_in': [
               'ext_svg/test1-tex.pdf_tex',
               'ext_svg/test2-0.9_svg-tex.pdf_tex',