)]}'
{"cinder/volume/flows/api/manage_existing.py":[{"author":{"_account_id":9648,"name":"Greg Hill","email":"greg.hill@rackspace.com","username":"jimbobhickville"},"change_message_id":"3bf1597506487c04be1f89f09d2aba552207e6a6","unresolved":false,"context_lines":[{"line_number":83,"context_line":"            \u0027volume\u0027: volume,"},{"line_number":84,"context_line":"        }"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def revert(self, context, result, optional_args\u003dNone, **kwargs):"},{"line_number":87,"context_line":"        # We never produced a result and therefore can\u0027t destroy anything."},{"line_number":88,"context_line":"        if isinstance(result, ft.Failure):"},{"line_number":89,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_7f2f5b3e","line":86,"updated":"2016-05-13 20:20:02.000000000","message":"Since `optional_args` isn\u0027t even used in this method, you *could* just remove it from the function signature.  Same effect. This works too, just seems weird to leave it behind if it isn\u0027t needed.","commit_id":"202caf2dedd7c7b8ed6c7790f125417228418ccb"}]}