)]}'
{"test/functional/swift.py":[{"author":{"_account_id":2696,"name":"Darrell Bishop","email":"dbishop@nvidia.com","username":"darrellb"},"change_message_id":"ba48054dae079e2d337e609436620eb89f5e282a","unresolved":false,"context_lines":[{"line_number":186,"context_line":"            quote \u003d urllib.quote"},{"line_number":187,"context_line":"            if cfg.get(\u0027no_quote\u0027) or cfg.get(\u0027no_path_quote\u0027):"},{"line_number":188,"context_line":"                quote \u003d lambda x: x"},{"line_number":189,"context_line":"            return \u0027%s/%s\u0027 % (self.storage_url, \u0027/\u0027.join([quote(i) for i"},{"line_number":190,"context_line":"                              in path]))"},{"line_number":191,"context_line":"        else:"},{"line_number":192,"context_line":"            return self.storage_url"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAALH%2F%2FgZo%3D","line":189,"updated":"2012-09-05 17:13:45.000000000","message":"This line would be more readable if you line-broke it after the comma (i.e. after \"...(self.storage_url,\").  It will still pass PEP8 and the whole \"X for Y in Z\" construct will be on the same line.","commit_id":"26d7f7c4c3f9e852336af95c0667fc758b61681d"},{"author":{"_account_id":2696,"name":"Darrell Bishop","email":"dbishop@nvidia.com","username":"darrellb"},"change_message_id":"ba48054dae079e2d337e609436620eb89f5e282a","unresolved":false,"context_lines":[{"line_number":209,"context_line":"            quote \u003d urllib.quote"},{"line_number":210,"context_line":"            if cfg.get(\u0027no_quote\u0027) or cfg.get(\u0027no_parms_quote\u0027):"},{"line_number":211,"context_line":"                quote \u003d lambda x: x"},{"line_number":212,"context_line":"            query_args \u003d [\u0027%s\u003d%s\u0027 % (quote(x), quote(str(y))) for (x, y) in"},{"line_number":213,"context_line":"                          parms.items()]"},{"line_number":214,"context_line":"            path \u003d \u0027%s?%s\u0027 % (path, \u0027\u0026\u0027.join(query_args))"},{"line_number":215,"context_line":"        if not cfg.get(\u0027no_content_length\u0027):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAALH%2F%2FgZY%3D","line":212,"updated":"2012-09-05 17:13:45.000000000","message":"I think this would be more readable with the line-break right before the \"for\".","commit_id":"26d7f7c4c3f9e852336af95c0667fc758b61681d"},{"author":{"_account_id":2696,"name":"Darrell Bishop","email":"dbishop@nvidia.com","username":"darrellb"},"change_message_id":"ba48054dae079e2d337e609436620eb89f5e282a","unresolved":false,"context_lines":[{"line_number":262,"context_line":"            quote \u003d urllib.quote"},{"line_number":263,"context_line":"            if cfg.get(\u0027no_quote\u0027) or cfg.get(\u0027no_parms_quote\u0027):"},{"line_number":264,"context_line":"                quote \u003d lambda x: x"},{"line_number":265,"context_line":"            query_args \u003d [\u0027%s\u003d%s\u0027 % (quote(x), quote(str(y))) for (x, y) in"},{"line_number":266,"context_line":"                          parms.items()]"},{"line_number":267,"context_line":"            path \u003d \u0027%s?%s\u0027 % (path, \u0027\u0026\u0027.join(query_args))"},{"line_number":268,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAALH%2F%2FgZQ%3D","line":265,"updated":"2012-09-05 17:13:45.000000000","message":"(ditto w/above)","commit_id":"26d7f7c4c3f9e852336af95c0667fc758b61681d"}]}