)]}'
{"swift/common/utils.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"97dbd6362885890f8cb36e1598499ff023245e6c","unresolved":false,"context_lines":[{"line_number":45,"context_line":"import glob"},{"line_number":46,"context_line":"from urlparse import urlparse as stdlib_urlparse, ParseResult"},{"line_number":47,"context_line":"import itertools"},{"line_number":48,"context_line":"import stat"},{"line_number":49,"context_line":"import eventlet"},{"line_number":50,"context_line":"import eventlet.semaphore"},{"line_number":51,"context_line":"from eventlet import GreenPool, sleep, Timeout, tpool, greenthread, \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5ao%3D","line":48,"updated":"2013-07-19 16:30:41.000000000","message":"Newline after system imports?","commit_id":"262a8c7b4dff967aaec9096f3b39a44416bc49a1"},{"author":{"_account_id":6198,"name":"Peter Portante","email":"peter.a.portante@gmail.com","username":"peter-a-portante"},"change_message_id":"10ac21f78825c787775addaafcb9c33c2cb67a8f","unresolved":false,"context_lines":[{"line_number":45,"context_line":"import glob"},{"line_number":46,"context_line":"from urlparse import urlparse as stdlib_urlparse, ParseResult"},{"line_number":47,"context_line":"import itertools"},{"line_number":48,"context_line":"import stat"},{"line_number":49,"context_line":"import eventlet"},{"line_number":50,"context_line":"import eventlet.semaphore"},{"line_number":51,"context_line":"from eventlet import GreenPool, sleep, Timeout, tpool, greenthread, \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5Tc%3D","line":48,"in_reply_to":"AAAAQn%2F%2F5ao%3D","updated":"2013-07-19 16:58:40.000000000","message":"Not sure I follow, but I\u0027ll play along.","commit_id":"262a8c7b4dff967aaec9096f3b39a44416bc49a1"}],"test/unit/common/middleware/test_recon.py":[{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"639c7a48f58d55b962277b8cbe860f381d0229b3","unresolved":false,"context_lines":[{"line_number":542,"context_line":"        unmounted_resp \u003d [{\u0027device\u0027: \u0027fakeone\u0027, \u0027mounted\u0027: False},"},{"line_number":543,"context_line":"                          {\u0027device\u0027: \u0027faketwo\u0027, \u0027mounted\u0027: False}]"},{"line_number":544,"context_line":"        self.mockos.ls_output\u003d[\u0027fakeone\u0027, \u0027faketwo\u0027]"},{"line_number":545,"context_line":"        self.mockos.ismount_output\u003dFalse"},{"line_number":546,"context_line":"        rv \u003d self.app.get_unmounted()"},{"line_number":547,"context_line":"        self.assertEquals(self.mockos.listdir_calls, [((\u0027/srv/node/\u0027,), {})])"},{"line_number":548,"context_line":"        self.assertEquals(rv, unmounted_resp)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F5EI%3D","line":545,"updated":"2013-07-19 17:41:27.000000000","message":"Spaces around the equal signs. (A few places)","commit_id":"c4bb509317fb100c55fedb99a608b5364149024d"},{"author":{"_account_id":6198,"name":"Peter Portante","email":"peter.a.portante@gmail.com","username":"peter-a-portante"},"change_message_id":"dbda55832b5cf5c73c560d6d5fa59c70f545a1aa","unresolved":false,"context_lines":[{"line_number":542,"context_line":"        unmounted_resp \u003d [{\u0027device\u0027: \u0027fakeone\u0027, \u0027mounted\u0027: False},"},{"line_number":543,"context_line":"                          {\u0027device\u0027: \u0027faketwo\u0027, \u0027mounted\u0027: False}]"},{"line_number":544,"context_line":"        self.mockos.ls_output\u003d[\u0027fakeone\u0027, \u0027faketwo\u0027]"},{"line_number":545,"context_line":"        self.mockos.ismount_output\u003dFalse"},{"line_number":546,"context_line":"        rv \u003d self.app.get_unmounted()"},{"line_number":547,"context_line":"        self.assertEquals(self.mockos.listdir_calls, [((\u0027/srv/node/\u0027,), {})])"},{"line_number":548,"context_line":"        self.assertEquals(rv, unmounted_resp)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F4%2F4%3D","line":545,"in_reply_to":"AAAAQn%2F%2F5EI%3D","updated":"2013-07-19 18:14:33.000000000","message":"Done","commit_id":"c4bb509317fb100c55fedb99a608b5364149024d"}],"test/unit/common/test_constraints.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"97dbd6362885890f8cb36e1598499ff023245e6c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import unittest"},{"line_number":17,"context_line":"import mock"},{"line_number":18,"context_line":"from test.unit import MockTrue"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from swift.common.swob import HTTPBadRequest, Request"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5aQ%3D","line":17,"updated":"2013-07-19 16:30:41.000000000","message":"Newline before froms?","commit_id":"262a8c7b4dff967aaec9096f3b39a44416bc49a1"},{"author":{"_account_id":6198,"name":"Peter Portante","email":"peter.a.portante@gmail.com","username":"peter-a-portante"},"change_message_id":"10ac21f78825c787775addaafcb9c33c2cb67a8f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import unittest"},{"line_number":17,"context_line":"import mock"},{"line_number":18,"context_line":"from test.unit import MockTrue"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from swift.common.swob import HTTPBadRequest, Request"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5TU%3D","line":17,"in_reply_to":"AAAAQn%2F%2F5aQ%3D","updated":"2013-07-19 16:58:40.000000000","message":"Wasn\u0027t one before, but I\u0027ll play.","commit_id":"262a8c7b4dff967aaec9096f3b39a44416bc49a1"}],"test/unit/common/test_utils.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"97dbd6362885890f8cb36e1598499ff023245e6c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"import time"},{"line_number":32,"context_line":"import unittest"},{"line_number":33,"context_line":"import fcntl"},{"line_number":34,"context_line":"import shutil"},{"line_number":35,"context_line":"from Queue import Queue, Empty"},{"line_number":36,"context_line":"from getpass import getuser"},{"line_number":37,"context_line":"from shutil import rmtree"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5aM%3D","line":34,"updated":"2013-07-19 16:30:41.000000000","message":"Newline before froms here too.","commit_id":"262a8c7b4dff967aaec9096f3b39a44416bc49a1"},{"author":{"_account_id":6198,"name":"Peter Portante","email":"peter.a.portante@gmail.com","username":"peter-a-portante"},"change_message_id":"10ac21f78825c787775addaafcb9c33c2cb67a8f","unresolved":false,"context_lines":[{"line_number":31,"context_line":"import time"},{"line_number":32,"context_line":"import unittest"},{"line_number":33,"context_line":"import fcntl"},{"line_number":34,"context_line":"import shutil"},{"line_number":35,"context_line":"from Queue import Queue, Empty"},{"line_number":36,"context_line":"from getpass import getuser"},{"line_number":37,"context_line":"from shutil import rmtree"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F5TM%3D","line":34,"in_reply_to":"AAAAQn%2F%2F5aM%3D","updated":"2013-07-19 16:58:40.000000000","message":"Sure, I\u0027ll play.","commit_id":"262a8c7b4dff967aaec9096f3b39a44416bc49a1"},{"author":{"_account_id":7680,"name":"Alex Gaynor","email":"alex.gaynor@gmail.com","username":"alex_gaynor"},"change_message_id":"639c7a48f58d55b962277b8cbe860f381d0229b3","unresolved":false,"context_lines":[{"line_number":1383,"context_line":"    def test_ismount_path_does_not_exist(self):"},{"line_number":1384,"context_line":"        tmpdir \u003d mkdtemp()"},{"line_number":1385,"context_line":"        try:"},{"line_number":1386,"context_line":"            assert False \u003d\u003d utils.ismount(os.path.join(tmpdir, \u0027bar\u0027))"},{"line_number":1387,"context_line":"        finally:"},{"line_number":1388,"context_line":"            shutil.rmtree(tmpdir)"},{"line_number":1389,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F5D0%3D","line":1386,"updated":"2013-07-19 17:41:27.000000000","message":"More idiomatic to just write `not utils.ismount(...)`. If you really want to check it\u0027s false, at least write `assert utils.ismount(...) is False`.","commit_id":"c4bb509317fb100c55fedb99a608b5364149024d"},{"author":{"_account_id":6198,"name":"Peter Portante","email":"peter.a.portante@gmail.com","username":"peter-a-portante"},"change_message_id":"dbda55832b5cf5c73c560d6d5fa59c70f545a1aa","unresolved":false,"context_lines":[{"line_number":1383,"context_line":"    def test_ismount_path_does_not_exist(self):"},{"line_number":1384,"context_line":"        tmpdir \u003d mkdtemp()"},{"line_number":1385,"context_line":"        try:"},{"line_number":1386,"context_line":"            assert False \u003d\u003d utils.ismount(os.path.join(tmpdir, \u0027bar\u0027))"},{"line_number":1387,"context_line":"        finally:"},{"line_number":1388,"context_line":"            shutil.rmtree(tmpdir)"},{"line_number":1389,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2F4%2F8%3D","line":1386,"in_reply_to":"AAAAQn%2F%2F5D0%3D","updated":"2013-07-19 18:14:33.000000000","message":"Done","commit_id":"c4bb509317fb100c55fedb99a608b5364149024d"}]}