)]}'
{"glance/api/v2/images.py":[{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"5d5d4951c9d7a988ccf20a631992a8824d5d96e0","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            pos \u003d int(path_pos)"},{"line_number":239,"context_line":"        elif path_pos !\u003d \u0027-\u0027:"},{"line_number":240,"context_line":"            return None"},{"line_number":241,"context_line":"        if (not allow_max) and (pos not in xrange(max_pos)):"},{"line_number":242,"context_line":"            return None"},{"line_number":243,"context_line":"        return pos"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_26a23a99","line":241,"updated":"2015-01-26 17:54:17.000000000","message":"Why is \"pos not in xrange\" used here? pos is a digit. This looks equivalent to\n\n    not (0 \u003c\u003d pos \u003c max_pos)\n\nThat would also improve the performance here. This whole statement should be rewritten as\n\n    if not (allow_max or 0 \u003c\u003d pos \u003c max_pos):","commit_id":"0b37a831ec3d3a5cf3638b312f0111400c962df7"},{"author":{"_account_id":11356,"name":"Louis Taylor","email":"louis@kragniz.eu","username":"kragniz"},"change_message_id":"e00f9437893c91f35132c57b99061aee5641a739","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            pos \u003d int(path_pos)"},{"line_number":239,"context_line":"        elif path_pos !\u003d \u0027-\u0027:"},{"line_number":240,"context_line":"            return None"},{"line_number":241,"context_line":"        if (not allow_max) and (pos not in xrange(max_pos)):"},{"line_number":242,"context_line":"            return None"},{"line_number":243,"context_line":"        return pos"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_3ae5ea37","line":241,"in_reply_to":"1a930d6b_26a23a99","updated":"2015-01-26 18:06:12.000000000","message":"+1","commit_id":"0b37a831ec3d3a5cf3638b312f0111400c962df7"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"def9d99d0804b78d1112b61696030136f00a56c0","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            pos \u003d int(path_pos)"},{"line_number":239,"context_line":"        elif path_pos !\u003d \u0027-\u0027:"},{"line_number":240,"context_line":"            return None"},{"line_number":241,"context_line":"        if (not allow_max) and (pos not in xrange(max_pos)):"},{"line_number":242,"context_line":"            return None"},{"line_number":243,"context_line":"        return pos"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a930d6b_b17ba1bc","line":241,"in_reply_to":"1a930d6b_26a23a99","updated":"2015-01-27 12:21:23.000000000","message":"This is indeed valid point.","commit_id":"0b37a831ec3d3a5cf3638b312f0111400c962df7"},{"author":{"_account_id":7487,"name":"hzrandd","email":"82433422@qq.com","username":"hzrandd"},"change_message_id":"6f1f5eae609f7e2deab8287c92d1f3318559b60d","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            pos \u003d int(path_pos)"},{"line_number":239,"context_line":"        elif path_pos !\u003d \u0027-\u0027:"},{"line_number":240,"context_line":"            return None"},{"line_number":241,"context_line":"        if (not allow_max) and (pos not in xrange(max_pos)):"},{"line_number":242,"context_line":"            return None"},{"line_number":243,"context_line":"        return pos"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa81d914_1e1d2f67","line":241,"in_reply_to":"1a930d6b_3ae5ea37","updated":"2015-01-28 03:04:58.000000000","message":"hzrandd\n\n+1","commit_id":"0b37a831ec3d3a5cf3638b312f0111400c962df7"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"34a77e13247bec168d8cf7b52cdd87c7795b41f2","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            pos \u003d int(path_pos)"},{"line_number":239,"context_line":"        elif path_pos !\u003d \u0027-\u0027:"},{"line_number":240,"context_line":"            return None"},{"line_number":241,"context_line":"        if not (allow_max or 0 \u003c\u003d pos \u003c max_pos):"},{"line_number":242,"context_line":"            return None"},{"line_number":243,"context_line":"        return pos"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba7be1f8_51a51b02","line":241,"updated":"2015-02-25 23:27:04.000000000","message":"Thank you!","commit_id":"a4f5bf6ab974c0d3fa1e0780298d6008589b14a8"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"a81c32ccd726282622955b5e9e7b50e6c90430ba","unresolved":false,"context_lines":[{"line_number":238,"context_line":"            pos \u003d int(path_pos)"},{"line_number":239,"context_line":"        elif path_pos !\u003d \u0027-\u0027:"},{"line_number":240,"context_line":"            return None"},{"line_number":241,"context_line":"        if not (allow_max or 0 \u003c\u003d pos \u003c max_pos):"},{"line_number":242,"context_line":"            return None"},{"line_number":243,"context_line":"        return pos"},{"line_number":244,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba7be1f8_61c32a38","line":241,"in_reply_to":"ba7be1f8_51a51b02","updated":"2015-02-25 23:31:56.000000000","message":"No, no, no, thank you! ;)","commit_id":"a4f5bf6ab974c0d3fa1e0780298d6008589b14a8"}]}