)]}'
{"neutron/common/exceptions.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"083a3fb3f01925b3d621edff0fb472beb30b0dc2","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        except Exception as e:"},{"line_number":39,"context_line":"            if _FATAL_EXCEPTION_FORMAT_ERRORS:"},{"line_number":40,"context_line":"                raise e"},{"line_number":41,"context_line":"            else:"},{"line_number":42,"context_line":"                # at least get the core message out if something happened"},{"line_number":43,"context_line":"                self._error_string \u003d self.message"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FPH8%3D","line":40,"updated":"2013-08-04 23:33:32.000000000","message":"I think this should just be \"raise\".","commit_id":"bf9d7a56e29567afb1c94f0315bbbffea41b9bf4"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"725b8727cf8c101580224a4358684633244b21b2","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        except Exception as e:"},{"line_number":39,"context_line":"            if _FATAL_EXCEPTION_FORMAT_ERRORS:"},{"line_number":40,"context_line":"                raise e"},{"line_number":41,"context_line":"            else:"},{"line_number":42,"context_line":"                # at least get the core message out if something happened"},{"line_number":43,"context_line":"                self._error_string \u003d self.message"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FN8c%3D","line":40,"in_reply_to":"AAAAQn%2F%2FPH8%3D","updated":"2013-08-05 10:14:13.000000000","message":"Done","commit_id":"bf9d7a56e29567afb1c94f0315bbbffea41b9bf4"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"083a3fb3f01925b3d621edff0fb472beb30b0dc2","unresolved":false,"context_lines":[{"line_number":195,"context_line":""},{"line_number":196,"context_line":"class Invalid(NeutronException):"},{"line_number":197,"context_line":"    def __init__(self, message\u003dNone):"},{"line_number":198,"context_line":"        self._error_string \u003d message"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"class InvalidInput(BadRequest):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FPHs%3D","line":198,"updated":"2013-08-04 23:33:32.000000000","message":"I feel like this should be setting self.message and calling the parent constructor.  At this point it makes no difference, but if the base exception class ever made a change to its constructor this class would miss it.","commit_id":"bf9d7a56e29567afb1c94f0315bbbffea41b9bf4"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"725b8727cf8c101580224a4358684633244b21b2","unresolved":false,"context_lines":[{"line_number":195,"context_line":""},{"line_number":196,"context_line":"class Invalid(NeutronException):"},{"line_number":197,"context_line":"    def __init__(self, message\u003dNone):"},{"line_number":198,"context_line":"        self._error_string \u003d message"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"class InvalidInput(BadRequest):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FN8Y%3D","line":198,"in_reply_to":"AAAAQn%2F%2FPHs%3D","updated":"2013-08-05 10:14:13.000000000","message":"Done","commit_id":"bf9d7a56e29567afb1c94f0315bbbffea41b9bf4"}]}