)]}'
{"swiftclient/service.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"5d1a647027f01554d370772bc71a2b5b6b5589bc","unresolved":false,"context_lines":[{"line_number":1763,"context_line":"                results_queue.put(res)"},{"line_number":1764,"context_line":"            return res"},{"line_number":1765,"context_line":"        finally:"},{"line_number":1766,"context_line":"            if fp is not None and hasattr(fp, \u0027close\u0027):"},{"line_number":1767,"context_line":"                fp.close()"},{"line_number":1768,"context_line":""},{"line_number":1769,"context_line":"    def _get_chunk_data(self, conn, container, obj, headers, manifest\u003dNone):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a1ff146_d74007a8","line":1766,"updated":"2017-03-16 16:15:11.000000000","message":"nit: given that fp came from open(), it should be safe to assume it has a close()","commit_id":"10483c91318c913eadc61c943f51ac72a45540a2"}],"tests/unit/test_service.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"5d1a647027f01554d370772bc71a2b5b6b5589bc","unresolved":false,"context_lines":[{"line_number":1117,"context_line":"                return contents.get_md5sum()"},{"line_number":1118,"context_line":""},{"line_number":1119,"context_line":"            mock_conn \u003d mock.Mock()"},{"line_number":1120,"context_line":"            mock_conn.put_object.side_effect \u003d _consuming_conn"},{"line_number":1121,"context_line":"            type(mock_conn).attempts \u003d mock.PropertyMock(return_value\u003d2)"},{"line_number":1122,"context_line":"            expected_r \u003d {"},{"line_number":1123,"context_line":"                \u0027action\u0027: \u0027upload_segment\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a1ff146_9745af71","line":1120,"updated":"2017-03-16 16:15:11.000000000","message":"Good call; nice bit of cleanup.","commit_id":"10483c91318c913eadc61c943f51ac72a45540a2"}]}
