Skip to content

Commit

Permalink
fix: remove impossible try/catch
Browse files Browse the repository at this point in the history
  • Loading branch information
dhdaines committed Mar 30, 2023
1 parent 9cd84d8 commit 2db239a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 2 additions & 0 deletions g2p/tests/test_transducer.py
Original file line number Diff line number Diff line change
Expand Up @@ -244,6 +244,8 @@ def test_normalize_edges(self):
bad_edges = [(0, None), (1, None), (2, 1)]
self.assertEqual(normalize_edges(bad_edges), [(0, 1), (1, 1), (2, 1)])
# Otherwise leave it as None
bad_edges = []
self.assertEqual(normalize_edges(bad_edges), bad_edges)
bad_edges = [(0, None)]
self.assertEqual(normalize_edges(bad_edges), bad_edges)
bad_edges = [(0, None), (1, None)]
Expand Down
6 changes: 2 additions & 4 deletions g2p/transducer/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,8 @@ def normalize_edges(
if edge[1] is None:
# if previous exists, use that, otherwise use following, otherwise None
previous = [x for x in edges[:i] if x[1] is not None]
try:
following = [x for x in edges[i + 1 :] if x[1] is not None]
except IndexError:
following = None
# Note: if len(edges) == 1, edges[1:] is empty, not an IndexError
following = [x for x in edges[i + 1 :] if x[1] is not None]
if previous:
edges[i] = (edge[0], previous[-1][1])
elif following:
Expand Down

0 comments on commit 2db239a

Please sign in to comment.