)]}'
{"swift/account/auditor.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"97a289a0d00a33d1f30645c890d0031d9181191d","unresolved":false,"context_lines":[{"line_number":125,"context_line":"                \u0027match the sum of %(key)s across policies (%(sum)s)\u0027)"},{"line_number":126,"context_line":"                % {\u0027key\u0027: key,"},{"line_number":127,"context_line":"                   \u0027total\u0027: info[key],"},{"line_number":128,"context_line":"                   \u0027sum\u0027: policy_totals[key]})"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def account_audit(self, path):"},{"line_number":131,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_d7383bf1","line":128,"updated":"2015-07-30 08:55:55.000000000","message":"this is a pep8 bug near as I can tell - it doesn\u0027t raise with pep8 1.6 - but their solution is questionable:\n\nhttps://github.com/PyCQA/pep8/pull/360\n\nUnfortunately them adding it *their* default_ignore doesn\u0027t help since we have custom exclusions/ignores in tox.ini.","commit_id":"4a987da8c816bc2d24b7dc6c31a5c72ea61aa441"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b3d0c5365e51e5923fedc8ce6e6984aab6287e60","unresolved":false,"context_lines":[{"line_number":125,"context_line":"                \u0027match the sum of %(key)s across policies (%(sum)s)\u0027)"},{"line_number":126,"context_line":"                % {\u0027key\u0027: key,"},{"line_number":127,"context_line":"                   \u0027total\u0027: info[key],"},{"line_number":128,"context_line":"                   \u0027sum\u0027: policy_totals[key]})"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    def account_audit(self, path):"},{"line_number":131,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_a2dbe87c","line":128,"in_reply_to":"3a50d1a3_d7383bf1","updated":"2015-07-30 09:56:02.000000000","message":"I\u0027m not sure that I understand your comment. My goal was to fix all E warnings, as asked by Alistair Coles, to not add \"regressions\" in term of pep8 checks for my patch https://review.openstack.org/#/c/205977/\n\nI know that they are many ways to write the same code and respect the PEP 8, but my goal here was only to fix \"tox -e pep8\". I have no preference for one style or for another.\n\nIf you prefer another style, please write directly the suggested style, I will update my patch ;-)","commit_id":"4a987da8c816bc2d24b7dc6c31a5c72ea61aa441"}],"swift/common/middleware/tempauth.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"97a289a0d00a33d1f30645c890d0031d9181191d","unresolved":false,"context_lines":[{"line_number":676,"context_line":"                    self.logger.increment(\u0027token_denied\u0027)"},{"line_number":677,"context_line":"                    auth \u003d \u0027Swift realm\u003d\"%s\"\u0027 % account"},{"line_number":678,"context_line":"                    return HTTPUnauthorized(request\u003dreq,"},{"line_number":679,"context_line":"                                            headers\u003d{\u0027Www-Authenticate\u0027: auth})"},{"line_number":680,"context_line":"                account2, user \u003d user.split(\u0027:\u0027, 1)"},{"line_number":681,"context_line":"                if account !\u003d account2:"},{"line_number":682,"context_line":"                    self.logger.increment(\u0027token_denied\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_ce91859c","line":679,"updated":"2015-07-30 08:55:55.000000000","message":"i found lots of other working solutions that didn\u0027t include an extra state/temp variable.","commit_id":"4a987da8c816bc2d24b7dc6c31a5c72ea61aa441"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b3d0c5365e51e5923fedc8ce6e6984aab6287e60","unresolved":false,"context_lines":[{"line_number":676,"context_line":"                    self.logger.increment(\u0027token_denied\u0027)"},{"line_number":677,"context_line":"                    auth \u003d \u0027Swift realm\u003d\"%s\"\u0027 % account"},{"line_number":678,"context_line":"                    return HTTPUnauthorized(request\u003dreq,"},{"line_number":679,"context_line":"                                            headers\u003d{\u0027Www-Authenticate\u0027: auth})"},{"line_number":680,"context_line":"                account2, user \u003d user.split(\u0027:\u0027, 1)"},{"line_number":681,"context_line":"                if account !\u003d account2:"},{"line_number":682,"context_line":"                    self.logger.increment(\u0027token_denied\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_a2c20885","line":679,"in_reply_to":"3a50d1a3_ce91859c","updated":"2015-07-30 09:56:02.000000000","message":"ditto","commit_id":"4a987da8c816bc2d24b7dc6c31a5c72ea61aa441"}],"swift/obj/reconstructor.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"97a289a0d00a33d1f30645c890d0031d9181191d","unresolved":false,"context_lines":[{"line_number":323,"context_line":"                              \u0027path\u0027: path,"},{"line_number":324,"context_line":"                              \u0027policy\u0027: policy,"},{"line_number":325,"context_line":"                              \u0027frag_index\u0027: frag_index,"},{"line_number":326,"context_line":"                          })"},{"line_number":327,"context_line":"                    break"},{"line_number":328,"context_line":"                if not all(fragment_payload):"},{"line_number":329,"context_line":"                    break"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_f1e540a4","side":"PARENT","line":326,"updated":"2015-07-30 08:55:55.000000000","message":"so also just dedent this line back two chars would have worked.\n\nI\u0027m trying to debug why vim-python-pep8-indent wants to indent here.","commit_id":"ffb11031502282e290dc50a49e6fc41f4a9512a3"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"b3d0c5365e51e5923fedc8ce6e6984aab6287e60","unresolved":false,"context_lines":[{"line_number":323,"context_line":"                              \u0027path\u0027: path,"},{"line_number":324,"context_line":"                              \u0027policy\u0027: policy,"},{"line_number":325,"context_line":"                              \u0027frag_index\u0027: frag_index,"},{"line_number":326,"context_line":"                          })"},{"line_number":327,"context_line":"                    break"},{"line_number":328,"context_line":"                if not all(fragment_payload):"},{"line_number":329,"context_line":"                    break"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_e23ed08a","side":"PARENT","line":326,"in_reply_to":"3a50d1a3_f1e540a4","updated":"2015-07-30 09:56:02.000000000","message":"ditto","commit_id":"ffb11031502282e290dc50a49e6fc41f4a9512a3"}],"test/unit/proxy/test_server.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"f53a9b7b67d0afb7faab6c617c5931108bb6661f","unresolved":false,"context_lines":[{"line_number":8498,"context_line":"            self.assert_status_map("},{"line_number":8499,"context_line":"                controller.POST,"},{"line_number":8500,"context_line":"                (404, 404, 404, 202, 202, 202, 201, 201, 201), 201)"},{"line_number":8501,"context_line":"            # account_info  PUT account  POST account"},{"line_number":8502,"context_line":"            self.assert_status_map("},{"line_number":8503,"context_line":"                controller.POST,"},{"line_number":8504,"context_line":"                (404, 404, 503, 201, 201, 503, 204, 204, 504), 204)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_97506cd6","line":8501,"updated":"2015-07-30 10:08:37.000000000","message":"i think the intent was for the comment to align with the response codes. moving it ahead of the response codes with original indentation works.\n\n  # account_info PUT account POST account\n  (404, 404, 404, 202, 202, 202, 201, 201, 201), 201)","commit_id":"389a97e3526df68121cecfaf1909343b5d381cbc"}]}