)]}'
{"openstackclient/compute/v2/security_group.py":[{"author":{"_account_id":13664,"name":"Amey Bhide","email":"abhide@vmware.com","username":"abhide"},"change_message_id":"e4cd89177ed23b15a2cbfaa279979a2ad6f237da","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        info[\u0027ip_range\u0027] \u003d info[\u0027ip_range\u0027][\u0027cidr\u0027]"},{"line_number":51,"context_line":"    else:"},{"line_number":52,"context_line":"        info[\u0027ip_range\u0027] \u003d \u0027\u0027"},{"line_number":53,"context_line":"    if info[\u0027ip_protocol\u0027].lower() \u003d\u003d \u0027icmp\u0027:"},{"line_number":54,"context_line":"        info[\u0027port_range\u0027] \u003d \u0027\u0027"},{"line_number":55,"context_line":"    elif info[\u0027ip_protocol\u0027] is None:"},{"line_number":56,"context_line":"        info[\u0027ip_protocol\u0027] \u003d \u0027\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_86ae0be6","line":53,"range":{"start_line":53,"start_character":13,"end_line":53,"end_character":24},"updated":"2015-07-23 23:28:42.000000000","message":"Might be better to move \nif info[\u0027ip_protocol\u0027] is None before if info[\u0027ip_protocol\u0027].lower() \n\nto avoid AttributeError\n\n\u003e\u003e\u003e x \u003d {}\n\u003e\u003e\u003e x[\u0027foo\u0027] \u003d None\n\u003e\u003e\u003e x[\u0027foo\u0027].lower()\nTraceback (most recent call last):\n  File \"\u003cstdin\u003e\", line 1, in \u003cmodule\u003e\nAttributeError: \u0027NoneType\u0027 object has no attribute \u0027lower\u0027","commit_id":"4530c75ed7dda2c86067433b14966a56f9367d0d"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f2c94c30841d681515c14897729828833dfd8732","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        info[\u0027ip_range\u0027] \u003d info[\u0027ip_range\u0027][\u0027cidr\u0027]"},{"line_number":51,"context_line":"    else:"},{"line_number":52,"context_line":"        info[\u0027ip_range\u0027] \u003d \u0027\u0027"},{"line_number":53,"context_line":"    if info[\u0027ip_protocol\u0027].lower() \u003d\u003d \u0027icmp\u0027:"},{"line_number":54,"context_line":"        info[\u0027port_range\u0027] \u003d \u0027\u0027"},{"line_number":55,"context_line":"    elif info[\u0027ip_protocol\u0027] is None:"},{"line_number":56,"context_line":"        info[\u0027ip_protocol\u0027] \u003d \u0027\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_19998c6e","line":53,"in_reply_to":"3a50d1a3_86ae0be6","updated":"2015-09-03 18:06:33.000000000","message":"yup...","commit_id":"4530c75ed7dda2c86067433b14966a56f9367d0d"}]}