)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"1467cbc9e77033dde21bcc62dc8885109d994601","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The code contains an incorrect int/tuple comparison during the binary"},{"line_number":10,"context_line":"search, which completely breaks the \"priority\" feature."},{"line_number":11,"context_line":"(Apparently python2 thinks int \u003c tuple is not an error and always True.)"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ic404b38eabc87b105c7c2443b6177071b26d97ce"},{"line_number":14,"context_line":"Closes-Bug: #1386946"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"5a890539_4231a0a1","line":11,"updated":"2014-11-08 22:28:14.000000000","message":"good catch... I always forget about this oddity","commit_id":"682f5a3effe544f0affabe384fcd3d4ee7d408f9"}],"neutron/plugins/bigswitch/tests/test_server.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"d87c46996edbb6ef83194145b88c557560fd0672","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1abeadc6_6d2c0b03","updated":"2014-08-28 20:46:02.000000000","message":"I\u0027m wondering why we even need this file.  Unless there is a reason to keep it, I\u0027d rather drop it.","commit_id":"59eb830c0950650602e88b8d31bb9625512d3974"},{"author":{"_account_id":11279,"name":"Angus Lees","email":"gus@inodes.org","username":"gus"},"change_message_id":"5d26ee757c95ddcf6ff18167a0fac60684bd34f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1abeadc6_85505b33","in_reply_to":"1abeadc6_6d2c0b03","updated":"2014-08-29 00:53:01.000000000","message":"I have nothing to do with the bigswitch plugin, but I like deleting code!","commit_id":"59eb830c0950650602e88b8d31bb9625512d3974"},{"author":{"_account_id":11279,"name":"Angus Lees","email":"gus@inodes.org","username":"gus"},"change_message_id":"12c65b04faaea691827eaba314d3197ae07a0e6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fa98f980_597ef4e5","in_reply_to":"1abeadc6_85505b33","updated":"2014-08-31 12:27:49.000000000","message":"Kevin says this is still needed, so I\u0027m restoring it and fixing it as per below.","commit_id":"59eb830c0950650602e88b8d31bb9625512d3974"},{"author":{"_account_id":6659,"name":"Paul Michali","email":"pc@michali.net","username":"pcm"},"change_message_id":"c8e64f7003e7b9949995d1bc88e9bfd92fe5d47f","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        retstatus \u003d self.default_status"},{"line_number":94,"context_line":"        retbody \u003d self.default_response"},{"line_number":95,"context_line":"        for i in moves.xrange(len(self.matches)):"},{"line_number":96,"context_line":"            (x, method_regexp, uri_regexp, handler, data, multi) \u003d \\"},{"line_number":97,"context_line":"                self.matches[i]"},{"line_number":98,"context_line":"            if re.match(method_regexp, method) and re.match(uri_regexp, uri):"},{"line_number":99,"context_line":"                kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_8a9b1f65","line":96,"updated":"2014-08-28 14:02:25.000000000","message":"Nit: Use parens for continuation, instead of backslash. Can use _, instead of x, for unused variable.","commit_id":"59eb830c0950650602e88b8d31bb9625512d3974"},{"author":{"_account_id":11279,"name":"Angus Lees","email":"gus@inodes.org","username":"gus"},"change_message_id":"5d26ee757c95ddcf6ff18167a0fac60684bd34f3","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        retstatus \u003d self.default_status"},{"line_number":94,"context_line":"        retbody \u003d self.default_response"},{"line_number":95,"context_line":"        for i in moves.xrange(len(self.matches)):"},{"line_number":96,"context_line":"            (x, method_regexp, uri_regexp, handler, data, multi) \u003d \\"},{"line_number":97,"context_line":"                self.matches[i]"},{"line_number":98,"context_line":"            if re.match(method_regexp, method) and re.match(uri_regexp, uri):"},{"line_number":99,"context_line":"                kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_45d563e3","line":96,"in_reply_to":"1abeadc6_8a9b1f65","updated":"2014-08-29 00:53:01.000000000","message":"No, _ is used for gettext translation no-op function in most openstack code.\n(Indeed much of one of my other changes was precisely changing mistaken uses of _ for \u0027x\u0027)\n\nI\u0027ll fix the backslash - or at least I would, except I\u0027m going with Mark\u0027s recommendation to just delete the entire file.","commit_id":"59eb830c0950650602e88b8d31bb9625512d3974"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"794664a734759213a808c63d489e2729000ce226","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        retstatus \u003d self.default_status"},{"line_number":92,"context_line":"        retbody \u003d self.default_response"},{"line_number":93,"context_line":"        for i in moves.xrange(len(self.matches)):"},{"line_number":94,"context_line":"            (unused_prior, method_regexp, uri_regexp, handler, data,"},{"line_number":95,"context_line":"             multi) \u003d self.matches[i]"},{"line_number":96,"context_line":"            if re.match(method_regexp, method) and re.match(uri_regexp, uri):"},{"line_number":97,"context_line":"                kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":11,"id":"5a890539_5c6ca435","line":94,"updated":"2014-11-13 23:04:03.000000000","message":"if you like you can just do _prior instead of unused prior, but I have no strong opinion","commit_id":"682f5a3effe544f0affabe384fcd3d4ee7d408f9"}]}