)]}'
{"nova/virt/libvirt/volume.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"102a4fe7415922018fc6a89a95d37714dd425c43","unresolved":false,"context_lines":[{"line_number":459,"context_line":"        if \u0027options\u0027 in connection_info[\u0027data\u0027]:"},{"line_number":460,"context_line":"            options \u003d connection_info[\u0027data\u0027][\u0027options\u0027]"},{"line_number":461,"context_line":"        else:"},{"line_number":462,"context_line":"            options \u003d None"},{"line_number":463,"context_line":"        path \u003d self._ensure_mounted(connection_info[\u0027data\u0027][\u0027export\u0027], options)"},{"line_number":464,"context_line":"        path \u003d os.path.join(path, connection_info[\u0027data\u0027][\u0027name\u0027])"},{"line_number":465,"context_line":"        conf.source_type \u003d \u0027file\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAO3%2F%2F9CU%3D","line":462,"updated":"2013-05-16 08:29:26.000000000","message":"Python idiom for this is connection_info[\u0027data\u0027].get(\u0027options\u0027)\n\nimho - way more readable.","commit_id":"b59685509e2ea188cbfce7f830858a7f69d5cb22"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"686897ed9056d6d6152445b16ca33c3928152229","unresolved":false,"context_lines":[{"line_number":459,"context_line":"        if \u0027options\u0027 in connection_info[\u0027data\u0027]:"},{"line_number":460,"context_line":"            options \u003d connection_info[\u0027data\u0027][\u0027options\u0027]"},{"line_number":461,"context_line":"        else:"},{"line_number":462,"context_line":"            options \u003d None"},{"line_number":463,"context_line":"        path \u003d self._ensure_mounted(connection_info[\u0027data\u0027][\u0027export\u0027], options)"},{"line_number":464,"context_line":"        path \u003d os.path.join(path, connection_info[\u0027data\u0027][\u0027name\u0027])"},{"line_number":465,"context_line":"        conf.source_type \u003d \u0027file\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAO3%2F%2F8gg%3D","line":462,"in_reply_to":"AAAAO3%2F%2F9CU%3D","updated":"2013-05-16 14:37:46.000000000","message":"I definitely agree.","commit_id":"b59685509e2ea188cbfce7f830858a7f69d5cb22"},{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"102a4fe7415922018fc6a89a95d37714dd425c43","unresolved":false,"context_lines":[{"line_number":582,"context_line":"        if \u0027options\u0027 in connection_info[\u0027data\u0027]:"},{"line_number":583,"context_line":"            options \u003d connection_info[\u0027data\u0027][\u0027options\u0027]"},{"line_number":584,"context_line":"        else:"},{"line_number":585,"context_line":"            options \u003d None"},{"line_number":586,"context_line":"        path \u003d self._ensure_mounted(connection_info[\u0027data\u0027][\u0027export\u0027], options)"},{"line_number":587,"context_line":"        path \u003d os.path.join(path, connection_info[\u0027data\u0027][\u0027name\u0027])"},{"line_number":588,"context_line":"        conf.source_type \u003d \u0027file\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAO3%2F%2F9CQ%3D","line":585,"updated":"2013-05-16 08:29:26.000000000","message":"Same as above","commit_id":"b59685509e2ea188cbfce7f830858a7f69d5cb22"}]}