)]}'
{"cinder/tests/test_emc.py":[{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"5b662d4ebd3eca4a089e89e6dfd36bb74e9632d2","unresolved":false,"context_lines":[{"line_number":335,"context_line":"        foundsync \u003d None"},{"line_number":336,"context_line":"        syncs \u003d self._enum_syncsvsvs()"},{"line_number":337,"context_line":"        for sync in syncs:"},{"line_number":338,"context_line":"            if sync[\u0027SyncedElement\u0027] \u003d\u003d objectpath[\u0027SyncedElement\u0027] and \\"},{"line_number":339,"context_line":"                    sync[\u0027SystemElement\u0027] \u003d\u003d sync[\u0027SystemElement\u0027]:"},{"line_number":340,"context_line":"                foundsync \u003d sync"},{"line_number":341,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F9ow%3D","line":338,"updated":"2013-03-05 05:46:36.000000000","message":"Use parentheses instead of line continuation operator\nhttp://www.python.org/dev/peps/pep-0008/#maximum-line-length","commit_id":"f240430e7408776010f3915cf7d76efced8789b1"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a07db8d098ce1defb52dbc40a211abbe7aa4fbdb","unresolved":false,"context_lines":[{"line_number":335,"context_line":"        foundsync \u003d None"},{"line_number":336,"context_line":"        syncs \u003d self._enum_syncsvsvs()"},{"line_number":337,"context_line":"        for sync in syncs:"},{"line_number":338,"context_line":"            if sync[\u0027SyncedElement\u0027] \u003d\u003d objectpath[\u0027SyncedElement\u0027] and \\"},{"line_number":339,"context_line":"                    sync[\u0027SystemElement\u0027] \u003d\u003d sync[\u0027SystemElement\u0027]:"},{"line_number":340,"context_line":"                foundsync \u003d sync"},{"line_number":341,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F8%2BE%3D","line":338,"in_reply_to":"AAAAN3%2F%2F9ow%3D","updated":"2013-03-05 13:58:50.000000000","message":"Done","commit_id":"f240430e7408776010f3915cf7d76efced8789b1"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"5b662d4ebd3eca4a089e89e6dfd36bb74e9632d2","unresolved":false,"context_lines":[{"line_number":336,"context_line":"        syncs \u003d self._enum_syncsvsvs()"},{"line_number":337,"context_line":"        for sync in syncs:"},{"line_number":338,"context_line":"            if sync[\u0027SyncedElement\u0027] \u003d\u003d objectpath[\u0027SyncedElement\u0027] and \\"},{"line_number":339,"context_line":"                    sync[\u0027SystemElement\u0027] \u003d\u003d sync[\u0027SystemElement\u0027]:"},{"line_number":340,"context_line":"                foundsync \u003d sync"},{"line_number":341,"context_line":"                break"},{"line_number":342,"context_line":"        return foundsync"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F9os%3D","line":339,"updated":"2013-03-05 05:46:36.000000000","message":"Do you mean \n sync[\u0027SystemElement\u0027] \u003d\u003d objectpath[\u0027SystemElement\u0027]?","commit_id":"f240430e7408776010f3915cf7d76efced8789b1"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a07db8d098ce1defb52dbc40a211abbe7aa4fbdb","unresolved":false,"context_lines":[{"line_number":336,"context_line":"        syncs \u003d self._enum_syncsvsvs()"},{"line_number":337,"context_line":"        for sync in syncs:"},{"line_number":338,"context_line":"            if sync[\u0027SyncedElement\u0027] \u003d\u003d objectpath[\u0027SyncedElement\u0027] and \\"},{"line_number":339,"context_line":"                    sync[\u0027SystemElement\u0027] \u003d\u003d sync[\u0027SystemElement\u0027]:"},{"line_number":340,"context_line":"                foundsync \u003d sync"},{"line_number":341,"context_line":"                break"},{"line_number":342,"context_line":"        return foundsync"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F8%2BI%3D","line":339,"in_reply_to":"AAAAN3%2F%2F9os%3D","updated":"2013-03-05 13:58:50.000000000","message":"Good catch.  Fixed.","commit_id":"f240430e7408776010f3915cf7d76efced8789b1"},{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"5b662d4ebd3eca4a089e89e6dfd36bb74e9632d2","unresolved":false,"context_lines":[{"line_number":519,"context_line":"        syncs \u003d []"},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"        vols \u003d self._enum_storagevolumes()"},{"line_number":522,"context_line":""},{"line_number":523,"context_line":"        objpath1 \u003d vols[0]"},{"line_number":524,"context_line":"        objpath2 \u003d vols[1]"},{"line_number":525,"context_line":"        sync \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F9oo%3D","line":522,"updated":"2013-03-05 05:46:36.000000000","message":"Looks like a lot of code duplication below, which caused you to update at so many locations. Can we extract out a method to do the task? Just a suggestion","commit_id":"f240430e7408776010f3915cf7d76efced8789b1"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"a07db8d098ce1defb52dbc40a211abbe7aa4fbdb","unresolved":false,"context_lines":[{"line_number":519,"context_line":"        syncs \u003d []"},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"        vols \u003d self._enum_storagevolumes()"},{"line_number":522,"context_line":""},{"line_number":523,"context_line":"        objpath1 \u003d vols[0]"},{"line_number":524,"context_line":"        objpath2 \u003d vols[1]"},{"line_number":525,"context_line":"        sync \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAN3%2F%2F8%2BM%3D","line":522,"in_reply_to":"AAAAN3%2F%2F9oo%3D","updated":"2013-03-05 13:58:50.000000000","message":"Done","commit_id":"f240430e7408776010f3915cf7d76efced8789b1"}],"cinder/volume/drivers/emc/emc_smis_common.py":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"3871708fd76cf58fac834bc0852bb37d7094a865","unresolved":false,"context_lines":[{"line_number":1139,"context_line":"            if vol_instance[\u0027ElementName\u0027] \u003d\u003d volumename:"},{"line_number":1140,"context_line":"                foundsyncname \u003d n"},{"line_number":1141,"context_line":"                storage_system \u003d vol_instance[\u0027SystemName\u0027]"},{"line_number":1142,"context_line":"                sync_instance \u003d self.conn.GetInstance(n, LocalOnly\u003dFalse)"},{"line_number":1143,"context_line":"                if waitforsync:"},{"line_number":1144,"context_line":"                    percent_synced \u003d sync_instance[\u0027PercentSynced\u0027]"},{"line_number":1145,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F%2FEM%3D","line":1142,"updated":"2013-03-04 17:10:54.000000000","message":"sync_instance is only used inside the if, so might as well move it inside and avoid excessive scope","commit_id":"cd8c7095f3ce44bda5746f08a5c0d3b2ae1036ad"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"49ec24028ac449eb130ba5d2cfcf1c2272bf4b0f","unresolved":false,"context_lines":[{"line_number":1139,"context_line":"            if vol_instance[\u0027ElementName\u0027] \u003d\u003d volumename:"},{"line_number":1140,"context_line":"                foundsyncname \u003d n"},{"line_number":1141,"context_line":"                storage_system \u003d vol_instance[\u0027SystemName\u0027]"},{"line_number":1142,"context_line":"                sync_instance \u003d self.conn.GetInstance(n, LocalOnly\u003dFalse)"},{"line_number":1143,"context_line":"                if waitforsync:"},{"line_number":1144,"context_line":"                    percent_synced \u003d sync_instance[\u0027PercentSynced\u0027]"},{"line_number":1145,"context_line":"                break"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F92w%3D","line":1142,"in_reply_to":"AAAAN3%2F%2F%2FEM%3D","updated":"2013-03-05 02:25:26.000000000","message":"Done","commit_id":"cd8c7095f3ce44bda5746f08a5c0d3b2ae1036ad"}]}