)]}'
{"tempest/common/rest_client.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ddf3682b729859ad89c199c3080ad94c0c6f9a48","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                             str(self.token)[0:STRING_LIMIT],"},{"line_number":94,"context_line":"                             str(self.headers)[0:STRING_LIMIT],"},{"line_number":95,"context_line":"                             str(self.general_header_lc)[0:STRING_LIMIT],"},{"line_number":96,"context_line":"                             str(self.response_header_lc)[0:STRING_LIMIT])"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    def _set_auth(self):"},{"line_number":99,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2FhGc%3D","line":96,"updated":"2013-09-25 11:29:59.000000000","message":"general_header_lc, response_header_lc are constants.\nDo you really want them in the __str__ ?","commit_id":"af340c803e9932cc0d2148a2fd6a878a1f6d83db"},{"author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"change_message_id":"c04e4c8fd698a716f8c5e3233cdf752c224ed01c","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                             str(self.token)[0:STRING_LIMIT],"},{"line_number":94,"context_line":"                             str(self.headers)[0:STRING_LIMIT],"},{"line_number":95,"context_line":"                             str(self.general_header_lc)[0:STRING_LIMIT],"},{"line_number":96,"context_line":"                             str(self.response_header_lc)[0:STRING_LIMIT])"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    def _set_auth(self):"},{"line_number":99,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAS3%2F%2Fgdg%3D","line":96,"in_reply_to":"AAAAS3%2F%2FhGc%3D","updated":"2013-09-25 14:54:23.000000000","message":"They are sets, afazekas. Are you suggesting to convert set to string, in the __str__?","commit_id":"af340c803e9932cc0d2148a2fd6a878a1f6d83db"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"92139c3d54c0818c08b3e2f2c90081583cfbc5e1","unresolved":false,"context_lines":[{"line_number":83,"context_line":"                     \"auth_url:%s, tenant_name:%s, auth_version:%s, \" + \\"},{"line_number":84,"context_line":"                     \"service:%s, base_url:%s, region:%s, \" + \\"},{"line_number":85,"context_line":"                     \"endpoint_url:%s, build_interval:%s, build_timeout:%s\" + \\"},{"line_number":86,"context_line":"                     \"\\ntoken:%s..., \\nheaders:%s...\""},{"line_number":87,"context_line":"        return str_format % (self.config, self.user, self.password,"},{"line_number":88,"context_line":"                             self.auth_url, self.tenant_name,"},{"line_number":89,"context_line":"                             self.auth_version, self.service,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FgLQ%3D","line":86,"updated":"2013-09-25 15:57:45.000000000","message":"I might be wrong here but in other projects where I usally review backslashes are verboten.\n\nYou should use parenthesis for multi-line statements e.g.:\n( \"config:%s, user:%s, password:%s, \"\n  \"auth_url:%s, tenant_name:%s, auth_version:%s, \"\n  \"service:%s, base_url:%s, region:%s, \"\n  \"endpoint_url:%s, build_interval:%s, build_timeout:%s\"\n  \"\\ntoken:%s..., \\nheaders:%s...\")","commit_id":"e63a54e926c77cbcaaa6d20f4e443158b2da6942"}]}