)]}'
{"freezer/job.py":[{"author":{"_account_id":27766,"name":"Matthias Bastian","email":"bastian@strato.de","username":"piepmatz"},"change_message_id":"a1cb727c0cf2cfb04d945ed87e3995a67effb542","unresolved":false,"context_lines":[{"line_number":97,"context_line":"            if self.conf.container:"},{"line_number":98,"context_line":"                container_name \u003d container.get(\u0027container_name\u0027)"},{"line_number":99,"context_line":"                if container_name \u003d\u003d self.conf.container:"},{"line_number":100,"context_line":"                    values \u003d ["},{"line_number":101,"context_line":"                        container.get(\u0027container_name\u0027),"},{"line_number":102,"context_line":"                        container.get(\u0027size\u0027),"},{"line_number":103,"context_line":"                        container.get(\u0027objects_count\u0027)"},{"line_number":104,"context_line":"                    ]"},{"line_number":105,"context_line":"                    data.append(values)"},{"line_number":106,"context_line":"                    break"},{"line_number":107,"context_line":"            else:"},{"line_number":108,"context_line":"                values \u003d ["}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_3932c0f3","line":105,"range":{"start_line":100,"start_character":20,"end_line":105,"end_character":39},"updated":"2018-04-11 09:20:09.000000000","message":"This is duplicated code.\n\nWhat about the following approach for the for loop?\n\nfor container in info:\n    if self.conf.container:\n        container_name \u003d container.get(\u0027container_name\u0027)\n        if container_name !\u003d self.conf.container:\n            continue\n\n    values \u003d [\n        container.get(\u0027container_name\u0027),\n        container.get(\u0027size\u0027),\n        container.get(\u0027objects_count\u0027)\n    ]\n    data.append(values)\n\n    if self.conf.container:\n        break  # values for given container were found","commit_id":"8edfdf777074699ce05d29600cb48d22c69c3dfe"}]}