Skip to content

Commit

Permalink
refactor: Remove unused variable (#611)
Browse files Browse the repository at this point in the history
- Fix potential double count of found subtitles
- Removed `subtitles_found` variable
  • Loading branch information
vagabondHustler authored Jan 28, 2024
1 parent d7fe92f commit b768c93
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/subsearch/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,6 @@ def download_files(self) -> None:
for enum, subtitle in enumerate(self.accepted_subtitles, 1):
io_file_system.download_subtitle(subtitle, enum, index_size)
self.downloaded_subtitles += 1
self.subtitles_found = index_size
io_log.log.task_completed()

@decorators.call_func
Expand All @@ -136,7 +135,6 @@ def download_manager(self) -> None:
subtitles = self.rejected_subtitles + self.accepted_subtitles
screen_manager.open_screen("download_manager", subtitles=subtitles)
self.manually_accepted_subtitles.extend(download_manager.DownloadManager.downloaded_subtitle)
self.subtitles_found += len(self.manually_accepted_subtitles)
io_log.log.task_completed()

@decorators.call_func
Expand Down

0 comments on commit b768c93

Please sign in to comment.