)]}'
{"glance/tests/unit/api/test_common.py":[{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"2efbef9ace9c186ec1567e1f58dc263f1a26c10f","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        while chunk:"},{"line_number":34,"context_line":"            yield chunk"},{"line_number":35,"context_line":"            chunk \u003d read_chunk()"},{"line_number":36,"context_line":"        else:"},{"line_number":37,"context_line":"            return"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_3d3e61da","line":36,"updated":"2018-08-08 17:24:20.000000000","message":"why not just remove the else condition and let it return naturally? Two lines less code :)","commit_id":"f537389811dcc5e0ec5ee02883f75a3baa34479e"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"eb75d23d58969c7ff76bcba29a9d0deb92aca896","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        while chunk:"},{"line_number":34,"context_line":"            yield chunk"},{"line_number":35,"context_line":"            chunk \u003d read_chunk()"},{"line_number":36,"context_line":"        else:"},{"line_number":37,"context_line":"            return"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_ce915948","line":36,"in_reply_to":"3f79a3b5_3d3e61da","updated":"2018-10-01 18:55:46.000000000","message":"+1","commit_id":"f537389811dcc5e0ec5ee02883f75a3baa34479e"}]}