)]}'
{"glance/registry/client/v1/client.py":[{"author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"change_message_id":"69b5270ca870aa480a33f644f8c461f91f0cb3c7","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        # store available passed request id for do_request call"},{"line_number":52,"context_line":"        if kwargs[\u0027request_id\u0027]:"},{"line_number":53,"context_line":"            self._passed_request_id \u003d kwargs[\u0027request_id\u0027]"},{"line_number":54,"context_line":"            del kwargs[\u0027request_id\u0027]"},{"line_number":55,"context_line":"        BaseClient.__init__(self, host, port, configure_via_auth\u003dFalse,"},{"line_number":56,"context_line":"                            **kwargs)"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dac4157f_d1731a14","line":54,"updated":"2015-04-22 07:43:41.000000000","message":"All of that could be replaced with single line\n\nself._passed_request_id \u003d kwargs.pop(\u0027request_id\u0027, None)\n\nThe effect will be a little bit different as the _passed_request_id is going to be always initialized (even if request_id is absent/empty), but hey - in the line 119 you are checking against self._passed_request_id which can fail with NameError right now.","commit_id":"ea76b77cc7b1b19c947e3e8bdf5abe0862942031"}]}