From df5fc374196f5b23a14104580bb9e511a8a4a7eb Mon Sep 17 00:00:00 2001 From: Jakob Moser Date: Sun, 1 Jun 2025 14:46:49 +0200 Subject: [PATCH] Don't generate no-ops --- karaokatalog/organize/move.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/karaokatalog/organize/move.py b/karaokatalog/organize/move.py index 1977853..9be26de 100644 --- a/karaokatalog/organize/move.py +++ b/karaokatalog/organize/move.py @@ -62,6 +62,12 @@ def move(songs: Sequence[Song], base_dir: Path) -> Sequence[MoveInstruction]: variant += 1 song_dir_strs_lower.add(str(canonical_song_dir).lower()) + + if song_dir == canonical_song_dir: + # After finding a free variant index, we arrived at the index we already had previously, this means + # we do not need to generate a move instruction + continue + move_instructions.append( MoveInstruction(absolute_song_dir, base_dir / canonical_song_dir) )