From 52dd3bc1c4235019158972037fe42dccfa75153c Mon Sep 17 00:00:00 2001 From: Juhani Haverinen Date: Sat, 8 Oct 2016 22:55:52 +0300 Subject: [PATCH] Fix a corner case where one time range was entirely contained with another, could lead to other overlaps not being noticed --- lukkari/__init__.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/lukkari/__init__.py b/lukkari/__init__.py index f9f239a..16b4e2a 100644 --- a/lukkari/__init__.py +++ b/lukkari/__init__.py @@ -72,4 +72,11 @@ def main(): if previous_time_range is not None: if time_range.overlaps(previous_time_range): print('\t\tOverlap') - previous_time_range = time_range + # If the current's ending time is before the previous's, don't change previous_time_range in case it overlaps with several + # If current's ending time after, it's safe to change + if time_range.range()[1] >= previous_time_range.range()[1]: + previous_time_range = time_range + else: + previous_time_range = time_range + else: + previous_time_range = time_range