)]}'
{"neutron/objects/network_segment_range.py":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"ebe252c2d7dd06b3a19f57b9d932d83b2b919f79","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                                     _filters[\u0027physical_network\u0027])"},{"line_number":194,"context_line":"            segment_ids \u003d set([])"},{"line_number":195,"context_line":"            for shared_range in shared_ranges.all():"},{"line_number":196,"context_line":"                segment_ids.update(set(range(shared_range.minimum,"},{"line_number":197,"context_line":"                                             shared_range.maximum + 1)))"},{"line_number":198,"context_line":"            if not segment_ids:"},{"line_number":199,"context_line":"                return []"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4c43b2_e717bb50","line":196,"range":{"start_line":196,"start_character":39,"end_line":196,"end_character":44},"updated":"2020-04-18 13:17:57.000000000","message":"No impact that this is not six_range here anymore? aka xrange for python2","commit_id":"62b23df20c44bdeb2fffad5497a8ecf7554c36ba"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c55ddb5aed24f7b47fe7c24e5d7eb1e68bce4978","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                                     _filters[\u0027physical_network\u0027])"},{"line_number":194,"context_line":"            segment_ids \u003d set([])"},{"line_number":195,"context_line":"            for shared_range in shared_ranges.all():"},{"line_number":196,"context_line":"                segment_ids.update(set(range(shared_range.minimum,"},{"line_number":197,"context_line":"                                             shared_range.maximum + 1)))"},{"line_number":198,"context_line":"            if not segment_ids:"},{"line_number":199,"context_line":"                return []"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_5645e3b2","line":196,"range":{"start_line":196,"start_character":39,"end_line":196,"end_character":44},"in_reply_to":"1f493fa4_22d736f0","updated":"2020-04-21 16:12:25.000000000","message":"BTW in python2, xrange is the same as range, but returns an iterator. In this case the result is the same.\n\nThis code section is being tested in the UTs (4 test cases, only \"test_get_segments_shared_no_shared_ranges\" does not reach this code)","commit_id":"62b23df20c44bdeb2fffad5497a8ecf7554c36ba"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"5374991ba3f1335c84b8beadaeab991b129144d2","unresolved":false,"context_lines":[{"line_number":193,"context_line":"                                     _filters[\u0027physical_network\u0027])"},{"line_number":194,"context_line":"            segment_ids \u003d set([])"},{"line_number":195,"context_line":"            for shared_range in shared_ranges.all():"},{"line_number":196,"context_line":"                segment_ids.update(set(range(shared_range.minimum,"},{"line_number":197,"context_line":"                                             shared_range.maximum + 1)))"},{"line_number":198,"context_line":"            if not segment_ids:"},{"line_number":199,"context_line":"                return []"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_22d736f0","line":196,"range":{"start_line":196,"start_character":39,"end_line":196,"end_character":44},"in_reply_to":"3f4c43b2_e717bb50","updated":"2020-04-21 15:51:56.000000000","message":"You are right! I need to use six-range","commit_id":"62b23df20c44bdeb2fffad5497a8ecf7554c36ba"}]}